-
Notifications
You must be signed in to change notification settings - Fork 182
Aligns Security Solution and Get Started pages #3812
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 |
lcawl
left a comment
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.
The inclusion of the project creation steps in the Getting Started with Security page LGTM
leemthompo
left a comment
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.
The 2 files dev docs "own" here we probably shouldn't, but looks fine to me in any case 😄
florent-leborgne
left a comment
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.
LGTM - ignore my suggestions if not relevant, those are more to try to stay consistent with a link for each item - I don't know if we have good matching content on single pages
Contributes to https://github.com/elastic/docs-team/issues/104 by making minor changes to the landing and get started pages.
Note
I'll add the redirects and files to delete in a subsequent PR.