-
Notifications
You must be signed in to change notification settings - Fork 162
AI Assistant chat sharing #2903
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
Conversation
🔍 Preview links for changed docs |
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.
Left a few suggestions for you, let me know if you have any questions, or feel free to ignore if I missed the mark. I think this is something that will make it's way to Observability at some point as well, so I'll make a note of that!
Co-authored-by: Mike Birnstiehl <[email protected]>
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.
aw, I love that a picture of my pup is making it into Kibana docs... she'll be so excited!
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.
Left a few small comments, but overall LGTM. Thanks for helping to get this out on time for next week's serverless release 🚀
Co-authored-by: Steph Milovic <[email protected]>
Co-authored-by: Steph Milovic <[email protected]>
…stic/docs-content into 2815-Assistant-convo-sharing
@benironside can we include screenshots of security topic conversations happening in each of our screenshots to better demonstrate use of the security AI Assistant? Thank you! |
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.
add screenshot which shows a security conversation in the background.
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.
Please add screenshot showing security conversation, add user's first and last name which will display in place of the email, add a friendly photo/avatar
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.
Please add screenshot showing security conversation, add user's first and last name which will display in place of the email, add a friendly photo/avatar
stack: ga 9.2 | ||
serverless: ga | ||
``` | ||
|
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.
can we talk a bit more about the "why" behind this functionality? like enabling collaboration within a SOC...etc?
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.
Left a few optional suggestions, otherwise LGTM!
Co-authored-by: Dhrumil Patel <[email protected]> Co-authored-by: natasha-moore-elastic <[email protected]>
creating a follow up issue to update the screenshots and add more "why" per Dhru's review |
Fixes #2815 by documenting chat sharing for AI Assistant. Related Kibana PR: elastic/kibana#230614