-
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
[Security Solution] [Cases] Cases UI Plugin for RAC #97646
Merged
Merged
Changes from 1 commit
Commits
Show all changes
87 commits
Select commit
Hold shift + click to select a range
756e5a5
init simple case ui plugin
michaelolo24 59756e8
remove hello world piece
michaelolo24 2b816e8
rename case to cases
michaelolo24 ebc609b
Merge 2b816e8c9e833ca2e2a361b29bc62861879982aa into 81da4f79e122b7c1b…
michaelolo24 d5cbcd1
move create
stephmilovic 841659b
clearing type errs
stephmilovic ead2da0
clearing type errs
stephmilovic 7a32042
resolve issue w exports
stephmilovic 359e8d1
its working
stephmilovic 83f6517
we are so good at this
stephmilovic 3ab3352
add mock
stephmilovic 51f6f4d
fixed components tests
stephmilovic 8f1e82d
flip USE_RAC_CASES_UI to false
stephmilovic 4763023
fix container tests
michaelolo24 249383f
Merge branch 'cases_moving_create' of https://github.com/stephmilovic…
michaelolo24 8f3424b
rm some comments
stephmilovic c1098f8
Merge branch 'cases_moving_create' of github.com:stephmilovic/kibana …
stephmilovic 7d76248
Merge branch 'master' into cases_moving_create
kibanamachine 4ff74fb
fix more imports
stephmilovic 87a1cde
Merge branch 'cases_moving_create' of github.com:stephmilovic/kibana …
stephmilovic 84714f9
fix more imports
stephmilovic 5f5021d
fix mistake
stephmilovic 731039a
add cases to kbn-optimizer/limits.yml
stephmilovic debbcb0
Fix i18n
stephmilovic 53e1ba9
Merge branch 'master' into cases_moving_create
stephmilovic 907bfc1
pr changes
stephmilovic 4f941e1
master merge
stephmilovic 82cd096
fix type and test
stephmilovic 1cb3917
fix a gosh darn silly type
stephmilovic 6d04fb3
one more Security remove
stephmilovic 46a801e
add prop for afterCaseCreated to make christos happiest boy in greece
stephmilovic 54ca62d
Merge branch 'master' into cases_rac_ui
stephmilovic 6b87eb0
[Security Solution][Cases] - Migrate all cases page (#94992)
michaelolo24 0d28c5a
[DOCS] Adds docs for search sessions (#94661)
gchaps 9ad1b49
[SECURITY_SOLUTION] Adds hash validation on UI for trusted app (#94958)
dasansol92 2e3f333
[uiSettings/theme] restrict theme options based on KBN_OPTIMIZER_THEM…
23cf913
[dashboard/bwc_shared_urls] check viz are loaded (#94941)
dmlemeshko f26b10c
[Security Solution] [Cases] Add deprecation in cases plugin for renam…
stephmilovic c3490d2
Merge branch 'master' into cases_rac_ui
stephmilovic 2c44d56
[Security Solution] [Cases] Move configure cases component (#95096)
stephmilovic bac342f
Merge branch 'master' into cases_rac_ui
stephmilovic d91bdf2
Merge branch 'master' into cases_rac_ui
stephmilovic 026c31c
[Security Solutions] [Cases] Update readme and add jsdocs to public m…
stephmilovic 47338bd
Merge branch 'master' into cases_rac_ui
stephmilovic 0187d83
trying to merge master
stephmilovic 191c58e
hoorah, fixed w master
stephmilovic 8826419
Merge branch 'master' into cases_rac_ui
michaelolo24 d8e4708
Merge branch 'master' into cases_rac_ui
stephmilovic 439d2e1
[Security Solution][Cases] Migrate case_details to cases plugin (#95086)
michaelolo24 cdd1b68
Merge branch 'master' into cases_rac_ui
stephmilovic ab5c6ee
Merge branch 'master' into cases_rac_ui
stephmilovic 442c57c
Merge branch 'master' into cases_rac_ui
stephmilovic bbabf7b
merge masteR
stephmilovic e0616b7
[Security Solution] [Cases] Deleting files from security solution and…
stephmilovic bc59d53
rm bazel dir
stephmilovic 8ba55b5
Merge branch 'master' into cases_rac_ui
stephmilovic 369e252
Merge branch 'master' into cases_rac_ui
stephmilovic 39f4844
[Security Solution] Tsconfig for cases (#96249)
stephmilovic 634b541
merge master
stephmilovic faee413
rm jsdom thirteen
stephmilovic ece4185
Merge branch 'master' into cases_rac_ui
stephmilovic 71a1b49
Merge branch 'master' into cases_rac_ui
michaelolo24 f2132d6
Merge branch 'master' into cases_rac_ui
michaelolo24 f1509f2
Merge branch 'master' into cases_rac_ui
michaelolo24 d2eaf3b
[Security Solution][Cases] - Update navigation props (#96493)
michaelolo24 47b4a0f
Merge branch 'master' into cases_rac_ui
stephmilovic 636524a
[Security Solution][Cases] Re-enable timeline functionality (#96496)
michaelolo24 85244ef
[Security Solution][Cases] Fix cases breadcrumbs (#97040)
michaelolo24 d7830be
Merge branch 'master' into cases_rac_ui
stephmilovic 76ae331
merge conflicts
stephmilovic fd7f810
fix bad push
stephmilovic 89721fa
fix type err
stephmilovic 64e35a1
Merge branch 'master' into cases_rac_ui
kibanamachine 54c97d4
[Security Solution] [Cases] All Cases Refactor & getAllCasesSelectorM…
stephmilovic 4d9bf46
readme changes
stephmilovic b9cdc2e
fix merge
stephmilovic 05aceaa
Merge branch 'master' into cases_rac_ui
stephmilovic 6cc459b
Merge branch 'master' into cases_rac_ui
stephmilovic 4b85082
pr changes 1
stephmilovic 870610c
ok undo
stephmilovic cc21d3a
[Security Solution] [Cases] Polishing up the jest tests in cases ui p…
stephmilovic b5d3932
fix double api calls
stephmilovic abf8cd9
Merge branch 'cases_rac_ui' of github.com:elastic/kibana into cases_r…
stephmilovic f5fa7f1
fix infinite loop, improve breadcrumb
michaelolo24 df7097e
fix attach alert to new case
michaelolo24 54d9730
Merge branch 'master' into cases_rac_ui
stephmilovic e0e4bee
rid of unnecessary log
stephmilovic 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
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.
Not sure if it's worth it or not (or if it causes a circular dependency) but we might be able to pull the
data
type from the case common types:https://github.com/elastic/kibana/blob/master/x-pack/plugins/cases/common/api/cases/comment.ts#L125
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.
Yea, was trying to avoid a circular dependency when I tossed this together, but thinking about it more, since cases doesn't depend on Security Solution, it shouldn't be