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

Link to the stable docs from v1 site #1849

Merged
merged 1 commit into from
Jan 7, 2022

Conversation

carolynvs
Copy link
Member

@carolynvs carolynvs commented Jan 6, 2022

What does this change

Makes it easier for people to know which version of the docs they are on (v0.38 or v1.0.0) and switch between the two. Long term, a better strategy for versioning the docs site would be great but this is a small change that we can do right now.

What issue does it fix

N/A

Notes for the reviewer

This PR has a buddy: See #1850.

Checklist

  • Unit Tests
  • Documentation
  • Schema (porter.yaml)

Makes it easier for people to know which version of the docs they are on (v0.38 or v1.0.0) and switch between the two. Long term, a better strategy for versioning the docs site would be great but this is a small change that we can do right now.

Signed-off-by: Carolyn Van Slyck <me@carolynvanslyck.com>
@carolynvs carolynvs mentioned this pull request Jan 6, 2022
3 tasks
@carolynvs
Copy link
Member Author

screenshot

The docs nav highlights that this is the 1.0 docs site and links to the stable (v0.38) docs.

Browser metadata
Path:      /docs/
Browser:   Firefox 92.0 on Mac OS 10.15
Viewport:  1440 x 677 @2x
Language:  en-US
Cookies:   Enabled

Open in BrowserStack

Open Deploy Preview · Mark as Resolved

@carolynvs carolynvs marked this pull request as ready for review January 6, 2022 15:47
@carolynvs carolynvs requested review from iennae and vdice and removed request for vdice and jeremyrickard January 6, 2022 15:47
@carolynvs carolynvs merged commit bef31c1 into getporter:release/v1 Jan 7, 2022
@carolynvs carolynvs deleted the link-main-docs branch January 7, 2022 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants