Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Filter system event definitions from content pack generation #21206

Merged
merged 2 commits into from
Dec 19, 2024

Conversation

janheise
Copy link
Contributor

@janheise janheise commented Dec 17, 2024

Description

Prior to this PR, a system event definition shows up in the list of exportable definitions for content packs. This leads to an error later on in content pack export. This PR filters the entities like with other definitions (anomaly etc.)

fixes #21166
solves customer issue

needs backport

Motivation and Context

How Has This Been Tested?

Tested manually by trying to create a content pack and look at the list of event definitions.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.

@janheise janheise requested review from a team, patrickmann and kingzacko1 December 17, 2024 09:49
Copy link
Contributor

@kingzacko1 kingzacko1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. System notification no longer listed in the list of exportable entities.

@janheise janheise merged commit 8aafad6 into master Dec 19, 2024
8 checks passed
@janheise janheise deleted the fix/filter_system_events_from_cp_generation branch December 19, 2024 08:14
garybot2 pushed a commit that referenced this pull request Dec 19, 2024
* filters system event definitions from content pack generation

* adding changelog

(cherry picked from commit 8aafad6)
garybot2 pushed a commit that referenced this pull request Dec 19, 2024
* filters system event definitions from content pack generation

* adding changelog

(cherry picked from commit 8aafad6)
garybot2 pushed a commit that referenced this pull request Dec 19, 2024
* filters system event definitions from content pack generation

* adding changelog

(cherry picked from commit 8aafad6)
danotorrey pushed a commit that referenced this pull request Jan 29, 2025
…#21220)

* filters system event definitions from content pack generation

* adding changelog

(cherry picked from commit 8aafad6)

Co-authored-by: Jan Heise <[email protected]>
danotorrey pushed a commit that referenced this pull request Jan 29, 2025
…#21221)

* filters system event definitions from content pack generation

* adding changelog

(cherry picked from commit 8aafad6)

Co-authored-by: Jan Heise <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove System Notification Events definition from list of exportable content pack entities
2 participants