Skip to content

Fix core layout #25025

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

Merged
merged 3 commits into from
Mar 3, 2023
Merged

Fix core layout #25025

merged 3 commits into from
Mar 3, 2023

Conversation

altmannyarden
Copy link
Contributor

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Description

Fixing an issue with layout where the dynamic section used the script GUID and not the script name.

Does it break backward compatibility?

  • Yes
    • Further details:
  • No

@altmannyarden
Copy link
Contributor Author

We will need to force merge to PR due to a false positive with the validation

[ERROR]: /home/circleci/project/Packs/Core/Layouts/layoutscontainer-Default.json: [GR103] - Content item 'Default' using content items: agent_ip_addresses,  which cannot be found in the repository.

[ERROR]: /home/circleci/project/Packs/Core/Layouts/layoutscontainer-Default_XDR_Alert.json: [GR103] - Content item 'Default XDR Alert' using content items:  which cannot be found in the repository.

@altmannyarden altmannyarden added the ForceMerge Forcing the merge of the PR despite the build status label Mar 3, 2023
@dantavori dantavori merged commit 65db24a into master Mar 3, 2023
@dantavori dantavori deleted the FixCoreLayout branch March 3, 2023 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ForceMerge Forcing the merge of the PR despite the build status
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants