-
Notifications
You must be signed in to change notification settings - Fork 191
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
ES|QL tab added to Timeline #4065
Conversation
Documentation previews: |
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.
These look great to me, thanks so much @nastasha-solomon !
@michaelolo24 some additional ES|QL docs were created this week, so I added some additional ES|QL resources and restructured some of the existing docs about ES|QL in Timeline. Can you take a look at this draft next week when you're back? Thanks! |
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.
Just a few attribute comments; LGTM otherwise 😄
Co-authored-by: Janeen Mikell Roberts <[email protected]>
Co-authored-by: Janeen Mikell Roberts <[email protected]>
Co-authored-by: Benjamin Ironside Goldstein <[email protected]> Co-authored-by: Abdon Pijpelink <[email protected]> Co-authored-by: Janeen Mikell Roberts <[email protected]> (cherry picked from commit 535a28c)
Co-authored-by: Benjamin Ironside Goldstein <[email protected]> Co-authored-by: Abdon Pijpelink <[email protected]> Co-authored-by: Janeen Mikell Roberts <[email protected]> Co-authored-by: Nastasha Solomon <[email protected]>
Fixes #4025
Preview: