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

fix(v1): v1 deploy preview should be available #3517

Merged
merged 2 commits into from
Oct 2, 2020

Conversation

slorber
Copy link
Collaborator

@slorber slorber commented Oct 1, 2020

Motivation

A simple v1 deploy preview should be available to review v1 PRs.

As we have multiple deploy previews now we can more easily include v1.

With only English, and only a minimal amount of version it does not increase deploy preview time significantly

@slorber slorber requested a review from lex111 as a code owner October 1, 2020 18:21
@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Oct 1, 2020
@docusaurus-bot
Copy link
Contributor

docusaurus-bot commented Oct 1, 2020

Deploy preview for docusaurus-2 ready!

Built with commit 308528e

https://deploy-preview-3517--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.

@slorber
Copy link
Collaborator Author

slorber commented Oct 1, 2020

Great :) plan to add the /v1-migrated deployment too so that we can improve the migration cli and easily review (likely MLH could work on this)

The scripts will need some refactoring though (as the Netlify site use v2 site as root dir, a bit confusing) but this can be done later

@slorber
Copy link
Collaborator Author

slorber commented Oct 2, 2020

New deploy preview for the v1 site migrated to v2 with the cli: https://deploy-preview-3517--docusaurus-2.netlify.app/v1-migrated/

Adds 90sec to Netlify deploy time, seems acceptable

@slorber slorber added the pr: bug fix This PR fixes a bug in a past release. label Oct 2, 2020
@slorber slorber merged commit e670473 into master Oct 2, 2020
@lex111 lex111 added this to the v2.0.0-alpha.65 milestone Oct 2, 2020
@lex111 lex111 deleted the slorber/v1-deploy-preview 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: bug fix This PR fixes a bug in a past release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants