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

MB-9609 migrate backend setup #30

Conversation

gidjin
Copy link
Contributor

@gidjin gidjin commented Oct 22, 2021

Moving the pages that belong under docs/backend/setup per the pattern. This PR is based on #24 and #27

@gidjin gidjin added the documentation Improvements or additions to documentation label Oct 22, 2021
@rogeruiz rogeruiz changed the base branch from main to rogeruiz/mb-9609-migrate-backend-pages October 22, 2021 12:19
@rogeruiz rogeruiz marked this pull request as draft October 22, 2021 12:20
@rogeruiz
Copy link
Contributor

Thank you @gidjin. 🎉

This looks good. I haven't had a chance to look at the commits directly yet but changed the base branch and the PR to Draft. There's still a few missing pages in the Guides section I need to migrate over.

Also I still need to add the redirects from the old routes to the new ones in the Docusaurus config.

Lastly, the order in which we merge these in should be #30, #27, #24.

@rogeruiz rogeruiz mentioned this pull request Oct 23, 2021
@rogeruiz rogeruiz force-pushed the rogeruiz/mb-9609-migrate-backend-pages branch from bdff1c9 to fb4e560 Compare October 23, 2021 03:28
@rogeruiz rogeruiz force-pushed the gidjin/MB-9609_migrate_backend_setup branch from c385a36 to 5a2c3a0 Compare October 23, 2021 03:50
@rogeruiz rogeruiz self-assigned this Oct 23, 2021
@rogeruiz rogeruiz marked this pull request as ready for review October 23, 2021 03:53
@rogeruiz rogeruiz changed the base branch from rogeruiz/mb-9609-migrate-backend-pages to rogeruiz/mb-9609-migrate-backend-guides-pages October 23, 2021 03:54
Copy link
Contributor

@rogeruiz rogeruiz left a comment

Choose a reason for hiding this comment

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

lgtm 🌈

@rogeruiz rogeruiz merged commit 0c1b5eb into rogeruiz/mb-9609-migrate-backend-guides-pages Oct 23, 2021
@rogeruiz rogeruiz deleted the gidjin/MB-9609_migrate_backend_setup branch October 23, 2021 03:55
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
Development

Successfully merging this pull request may close these issues.

2 participants