-
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
Creates "AI for security" section #5443
Conversation
A documentation preview will be available soon. Request a new doc build by commenting
If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here. |
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.
A few suggestions to consider, otherwise looks good!
include::azure-openai-setup.asciidoc[leveloffset=+2] | ||
include::connect-to-openai.asciidoc[leveloffset=+2] | ||
include::connect-to-bedrock.asciidoc[leveloffset=+2] | ||
|
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.
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.
🚀
* first test of new org * fixes build error * updates landing pages * incorporates Nat's review (cherry picked from commit d32f517)
* first test of new org * fixes build error * updates landing pages * incorporates Nat's review (cherry picked from commit d32f517) Co-authored-by: Benjamin Ironside Goldstein <[email protected]>
Fixes #5442
This PR touched a lot of files, but most of the changes in the diff are due to updated folder/section names ("AI Assistant" became "AI for security").
Preview: AI for Security