-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[RAM] Implement global alerts page #175143
Merged
umbopepato
merged 48 commits into
elastic:main
from
umbopepato:173647-global-alerts-page
Feb 13, 2024
Merged
Changes from all commits
Commits
Show all changes
48 commits
Select commit
Hold shift + click to select a range
b373792
WIP
umbopepato 1ca128d
Merge branch 'main' of github.com:elastic/kibana into 173647-global-a…
umbopepato 5ca182e
Draft basic global alerts page
umbopepato bc455f8
Improve global alerts page composition and filtering, move to correct…
umbopepato 73e4212
Improve global alerts page, add tests
umbopepato b4c2b4c
Merge branch 'main' of github.com:elastic/kibana into 173647-global-a…
umbopepato c943197
Merge branch 'main' of github.com:elastic/kibana into 173647-global-a…
umbopepato 56363bd
Remove unused translations
umbopepato cf8b85c
Fix type errors
umbopepato 428be5e
Merge branch 'main' of github.com:elastic/kibana into 173647-global-a…
umbopepato fed992b
Set default global alerts page time filter to last 24H
umbopepato cf338c8
Remove remaining internal alerts page references
umbopepato 1799015
Remove unused translations
umbopepato a0ae54c
Merge branch 'main' into 173647-global-alerts-page
umbopepato f1f4265
Remove test import of deleted test file
umbopepato 3162700
Merge branch 'main' of github.com:elastic/kibana into 173647-global-a…
umbopepato aad3409
Merge branch '173647-global-alerts-page' of github.com:umbopepato/kib…
umbopepato cdcdcf9
Merge branch 'main' into 173647-global-alerts-page
umbopepato 6348918
Align new alerts table columns parameters to spec
umbopepato b35a559
Merge branch 'main' of github.com:elastic/kibana into 173647-global-a…
umbopepato 41c585c
Merge branch 'main' into 173647-global-alerts-page
umbopepato 865e958
Add missing monitoring alerts feature id to global alerts page
umbopepato 9359c28
Fix alerts feature id detection logic and rule types grouping
umbopepato 029a525
Fix global alerts page time filter not working with security alerts
umbopepato 8090de3
Fix global alerts table id and prevent columns misalignment
umbopepato 4fe65cd
Improve quick filters clarity conversion clarity
umbopepato 6cd7612
Show error toast for potential alerts search bar query parsing errors
umbopepato 63660dc
Fix regression that caused unwanted alerts search bar state resets
umbopepato 259c2b1
Merge branch 'main' of github.com:elastic/kibana into 173647-global-a…
umbopepato eb7daae
Add tech preview badge to global alerts page
umbopepato d5a0c1e
Merge branch '173647-global-alerts-page' of github.com:umbopepato/kib…
umbopepato bfbaa93
Improve alerts search bar state synchrinzation logic
umbopepato da37428
Merge branch 'main' of github.com:elastic/kibana into 173647-global-a…
umbopepato 51c4673
Fix global alerts page functional test
umbopepato a75223b
Merge branch 'main' of github.com:elastic/kibana into 173647-global-a…
umbopepato 876d9ab
Add quick filters story to search bar Storybook
umbopepato a10e890
Merge branch 'main' of github.com:elastic/kibana into 173647-global-a…
umbopepato 35b8fce
Merge branch 'main' of github.com:elastic/kibana into 173647-global-a…
umbopepato 3bdf33a
Extract quick filters functionality out of unified search bar
umbopepato 2030bce
Merge branch 'main' of github.com:elastic/kibana into 173647-global-a…
umbopepato 712a86b
Separate items and panels in additionalQueryBarMenuItems
umbopepato 07bea51
Merge branch 'main' of github.com:elastic/kibana into 173647-global-a…
umbopepato d93205e
Update additional query bar menu items storybook
umbopepato 9afc692
Fix type error
umbopepato fa603a4
Merge branch 'main' of github.com:elastic/kibana into 173647-global-a…
umbopepato 42bbd4a
Fix additional query bar menu items test
umbopepato a6b0fce
Merge branch 'main' of github.com:elastic/kibana into 173647-global-a…
umbopepato cf31c41
Merge branch 'main' of github.com:elastic/kibana into 173647-global-a…
umbopepato File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -32,7 +32,7 @@ | |
"features", | ||
"home", | ||
"spaces", | ||
"serverless", | ||
"serverless" | ||
], | ||
"requiredBundles": [ | ||
"alerting", | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Wow, we really don't have a type guard function for this already?
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.
I was surprised too, apparently many plugins/packages have slightly different variants of this... I couldn't find a generic module from where to import it so I ended up adding our variant but if you have a better solution in mind please let me know 🙂