-
Notifications
You must be signed in to change notification settings - Fork 8k
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
[Security Solution][Alert details] - extract panel keys to constant file to avoid weird webpack issue (circular dependency?) #181894
[Security Solution][Alert details] - extract panel keys to constant file to avoid weird webpack issue (circular dependency?) #181894
Conversation
Pinging @elastic/security-threat-hunting-investigations (Team:Threat Hunting:Investigations) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
a22bf3f
to
bd93aca
Compare
…ile to avoid weird webpack issue (circular dependency?)
bd93aca
to
3ec29d8
Compare
💚 Build Succeeded
Metrics [docs]Module Count
Async chunks
Page load bundle
History
To update your PR or re-run it, just comment with: |
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…ile to avoid weird webpack issue (circular dependency?) (elastic#181894) (cherry picked from commit 0a9ad29) # Conflicts: # x-pack/plugins/security_solution/public/flyout/document_details/right/components/highlighted_fields_cell.test.tsx
x-pack/plugins/security_solution/public/flyout/document_details/shared/constants/panel_keys.ts
Show resolved
Hide resolved
…stant file to avoid weird webpack issue (circular dependency?) (#181894) (#181929) # Backport This will backport the following commits from `main` to `8.14`: - [[Security Solution][Alert details] - extract panel keys to constant file to avoid weird webpack issue (circular dependency?) (#181894)](#181894) <!--- Backport version: 8.9.8 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Philippe Oberti","email":"philippe.oberti@elastic.co"},"sourceCommit":{"committedDate":"2024-04-26T22:00:34Z","message":"[Security Solution][Alert details] - extract panel keys to constant file to avoid weird webpack issue (circular dependency?) (#181894)","sha":"0a9ad2956ebb77829ec860d8511d74244aceddc8","branchLabelMapping":{"^v8.15.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Threat Hunting:Investigations","v8.14.0","v8.15.0"],"number":181894,"url":"#181894 Solution][Alert details] - extract panel keys to constant file to avoid weird webpack issue (circular dependency?) (#181894)","sha":"0a9ad2956ebb77829ec860d8511d74244aceddc8"}},"sourceBranch":"main","suggestedTargetBranches":["8.14"],"targetPullRequestStates":[{"branch":"8.14","label":"v8.14.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.15.0","labelRegex":"^v8.15.0$","isSourceBranch":true,"state":"MERGED","url":"#181894 Solution][Alert details] - extract panel keys to constant file to avoid weird webpack issue (circular dependency?) (#181894)","sha":"0a9ad2956ebb77829ec860d8511d74244aceddc8"}}]}] BACKPORT-->
…ile to avoid weird webpack issue (circular dependency?) (elastic#181894)
Summary
While making a very small change to the note_previews component (see pending PR), I noticed some jest tests failing. Those tests seem to be completely unrelated to the changes made in the PR... After investigation, the issues comes from the
DocumentDetailsRightPanelKey
constant that we import. There must be some sort of circular dependency, or something Webpack isn't happy about...This current PR only extracts those constants to a separate file. This fixes the jest tests failing in the PR mentioned above.
Absolutely no code changes are implemented, other than extracting the constants and updating all the imports.
Will help merge this, so indirectly addresses this bug