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

chore(v2): v2 website should make it easy to contribute to upstream docs #3506

Merged
merged 2 commits into from
Oct 1, 2020

Conversation

slorber
Copy link
Collaborator

@slorber slorber commented Sep 30, 2020

Motivation

By default, in prod, the "latest version" is the last release, not "next".

This lead to the editUrl targeting docs of a specific version, not upstream docs.

This leads to maintainers asking to port the doc fixes to upstream docs, and sometimes merging by mistake to the versioned docs, so the doc fix would disappear.

Here is an example: #3428 (comment)

By defaulting to the next version in prod, we nudge contributors and maintainers to contribute to upstream docs in priority.

image

The label (unreleased) can help make it obvious that it's the doc for an unreleased version, so that the user can switch if needed.

But anyway, between one alpha to another, the docs do not change that much, so I think this new behavior is fine for the whole alpha phase.

@lex111 @yangshun any opinion?

… docs the main docs so that editUrl will modify upstream docs in priority
@slorber slorber requested a review from yangshun as a code owner September 30, 2020 18:20
@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Sep 30, 2020
@slorber slorber added pr: maintenance This PR does not produce any behavior differences to end users when upgrading. and removed CLA Signed Signed Facebook CLA labels Sep 30, 2020
@docusaurus-bot
Copy link
Contributor

docusaurus-bot commented Sep 30, 2020

Deploy preview for docusaurus-2 ready!

Built with commit a618141

https://deploy-preview-3506--docusaurus-2.netlify.app

Copy link
Contributor

@lex111 lex111 left a comment

Choose a reason for hiding this comment

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

I like this idea, so we need to replace 'next' label with 'unreleased' one, right?

@slorber
Copy link
Collaborator Author

slorber commented Oct 1, 2020

@slorber slorber added the CLA Signed Signed Facebook CLA label Oct 1, 2020
Copy link
Contributor

@lex111 lex111 left a comment

Choose a reason for hiding this comment

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

LGTM

@slorber slorber merged commit 85ff9bd into master Oct 1, 2020
@slorber
Copy link
Collaborator Author

slorber commented Oct 1, 2020

erf there's the banner here that is annoying now 😅

image

will try to fix it as part of #3013

@lex111 lex111 added this to the v2.0.0-alpha.65 milestone Oct 2, 2020
@lex111 lex111 deleted the slorber/use-last-version-current branch October 7, 2020 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Signed Facebook CLA pr: maintenance This PR does not produce any behavior differences to end users when upgrading.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants