Skip to content

Conversation

@PhillypHenning
Copy link
Contributor

@PhillypHenning PhillypHenning commented Aug 10, 2022

Quick update to splash text

@arm4b arm4b requested review from ccapell and mickmcgrath13 August 10, 2022 20:26
Copy link
Contributor

@arm4b arm4b left a comment

Choose a reason for hiding this comment

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

Looks much more clear to me 👍

@arm4b
Copy link
Contributor

arm4b commented Aug 10, 2022

@mickmcgrath13 @ccapell WDYT? ^^

We went to the docs and it looked confusing:
image
Now I see that Consistently Deploy is a separated <h1>, though the new version by @PhillypHenning looks like a better description.

Copy link
Contributor

@ccapell ccapell left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@armab, you won't see the changes in the documentation site until they are approved and pushed.

@arm4b arm4b added documentation 📗 Improvements or additions to documentation enhancement ✨ New feature or request labels Aug 10, 2022
@arm4b
Copy link
Contributor

arm4b commented Aug 11, 2022

Nice! Looks like we're ready to get this merged @PhillypHenning with all the consensus 👍

@PhillypHenning PhillypHenning merged commit 2dac857 into main Aug 11, 2022
@PhillypHenning PhillypHenning deleted the index-update branch August 11, 2022 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation 📗 Improvements or additions to documentation enhancement ✨ New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants