Skip to content
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

[DOCS] Update screenshot of the "Getting started" page #2049

Merged
merged 4 commits into from
Jun 16, 2022

Conversation

nastasha-solomon
Copy link
Contributor

@nastasha-solomon nastasha-solomon commented Jun 7, 2022

Fixes #1864.

Preview here.

@nastasha-solomon
Copy link
Contributor Author

nastasha-solomon commented Jun 8, 2022

@stephmilovic @jmikell821 should I adjust the page description mention that we also offer cloud protection? Or do you think the first sentence is fine as is?
Screen Shot 2022-06-08 at 5 10 34 PM

@jmikell821
Copy link
Contributor

@nastasha-solomon I think it's fine but we may want to get another opinion just in case.

@nastasha-solomon nastasha-solomon requested review from stephmilovic and a team June 13, 2022 20:08
@nastasha-solomon nastasha-solomon marked this pull request as ready for review June 13, 2022 20:08
Copy link
Contributor

@joepeeples joepeeples left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshot LGTM, but it looks like in 8.3.0 the page was renamed to "Get started", for both legacy and new/grouped navs:

image

image

@nastasha-solomon
Copy link
Contributor Author

Good catch, thanks @joepeeples !

@nastasha-solomon nastasha-solomon mentioned this pull request Jun 13, 2022
15 tasks
@nastasha-solomon nastasha-solomon merged commit dc57d45 into main Jun 16, 2022
@nastasha-solomon nastasha-solomon deleted the issue-1864-updt-gs-page branch June 16, 2022 20:00
mergify bot pushed a commit that referenced this pull request Jun 16, 2022
nastasha-solomon added a commit that referenced this pull request Jun 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOCS] Update screenshot of the "Getting started" page
3 participants