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
Add date picker in timeline + clean up queryDate in draggable wrapper… #35340
Conversation
XavierM
commented
Apr 19, 2019
- Add super date picker on Timeline
- Fix load more on Timeline
- Clean up all the poll from the old way to refresh data to avoid multiple queries
- more clean of files of components that we do not use anymore, no need to be nostalgic
Pinging @elastic/secops |
💔 Build Failed |
ff33688
to
662834f
Compare
💔 Build Failed |
662834f
to
16c0339
Compare
💔 Build Failed |
x-pack/plugins/siem/public/components/timeline/properties/index.tsx
Outdated
Show resolved
Hide resolved
c847a5b
to
37badf0
Compare
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.
This well executed feature resolves the ambiguity around date ranges in the timeline, and sets us up nicely for timeline persistence, which requires previously-opened timelines to be de-coupled (by default), from the global date range when they are opened. LGTM 🚀
… + remove all the poll stuff
💔 Build Failed |
37badf0
to
e11a32a
Compare
💚 Build Succeeded |
💚 Build Succeeded |
x-pack/plugins/siem/public/components/localized_date_tooltip/index.tsx
Outdated
Show resolved
Hide resolved
x-pack/test/functional/apps/advanced_settings/feature_controls/advanced_settings_security.ts
Show resolved
Hide resolved
x-pack/plugins/siem/public/components/page/overview/overview_host/index.tsx
Show resolved
Hide resolved
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.
Checked out and tested locally along with code review. Thorough cleanup + refactoring of QueryDate
, plus new timeline datepicker & syncing feature is ++ Thanks for the attention to detail Xavier! LGTM!
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.
Thanks for the date time tethering, the cleanup of redux, and the extra CI build changes to make us as close to master as possible.
Out of my top 10 LGTM's, this scores in my top 5.
💔 Build Failed |
3ae2236
to
ecdccfb
Compare
💚 Build Succeeded |