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

docs: fix i18n routes to feature requests #5843

Merged
merged 2 commits into from Oct 31, 2021
Merged

Conversation

Josh-Cena
Copy link
Collaborator

Motivation

Fix #5553. Append base URL to routes so that i18n routes are created as well.

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

https://docusaurus.io/zh-CN//feature-requests

@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Oct 31, 2021
@Josh-Cena Josh-Cena added the pr: documentation This PR works on the website or other text documents in the repo. label Oct 31, 2021
@netlify
Copy link

netlify bot commented Oct 31, 2021

✔️ [V2]

🔨 Explore the source changes: 5196859

🔍 Inspect the deploy log: https://app.netlify.com/sites/docusaurus-2/deploys/617e28dc8cfc750007e161d0

😎 Browse the preview: https://deploy-preview-5843--docusaurus-2.netlify.app

@github-actions
Copy link

github-actions bot commented Oct 31, 2021

⚡️ Lighthouse report for the changes in this PR:

Category Score
🟢 Performance 95
🟢 Accessibility 98
🟢 Best practices 100
🟢 SEO 100
🟢 PWA 95

Lighthouse ran on https://deploy-preview-5843--docusaurus-2.netlify.app/

@github-actions
Copy link

github-actions bot commented Oct 31, 2021

Size Change: -12 B (0%)

Total Size: 847 kB

ℹ️ View Unchanged
Filename Size Change
website/.docusaurus/globalData.json 38 kB 0 B
website/build/assets/css/styles.********.css 94 kB 0 B
website/build/assets/js/main.********.js 425 kB 0 B
website/build/blog/2017/12/14/introducing-docusaurus/index.html 68.2 kB 0 B
website/build/blog/index.html 39 kB 0 B
website/build/docs/index.html 46.1 kB -6 B (0%)
website/build/docs/installation/index.html 53.6 kB -6 B (0%)
website/build/index.html 31.6 kB 0 B
website/build/tests/docs/index.html 27.3 kB 0 B
website/build/tests/docs/standalone/index.html 23.8 kB 0 B

compressed-size-action

@Josh-Cena Josh-Cena merged commit 1e1df74 into main Oct 31, 2021
@Josh-Cena Josh-Cena deleted the jc/fix-feature-request branch October 31, 2021 05:32
wpyoga pushed a commit to wpyoga/docusaurus that referenced this pull request Oct 31, 2021
* docs: fix i18n routes to feature requests

* Add redirect rules
Josh-Cena added a commit that referenced this pull request Nov 10, 2021
* feat: allow GIT_USER env var to be unset if SSH is used

* fix: packages/docusaurus/src/commands/deploy.ts

Co-authored-by: Joshua Chen <sidachen2003@gmail.com>

* feat: allow user to specify deploymentBranch property in docusaurus.config.js (#5841)

* feat: allow user to specify deploymentBranch property in docusaurus.config.js

* docs: remove extra backtick

* docs: fix broken code block

* docs: fix i18n routes to feature requests (#5843)

* docs: fix i18n routes to feature requests

* Add redirect rules

* feat: allow GIT_USER env var to be unset if SSH is used

* fix: packages/docusaurus/src/commands/deploy.ts

Co-authored-by: Joshua Chen <sidachen2003@gmail.com>

* fix: avoid escaping hyphen in regex

* Refactor

* Update deployment.mdx

* Make SSH higher priority

* Only infer but not override

* Add tests

* Fix tests

* Fix

Co-authored-by: Joshua Chen <sidachen2003@gmail.com>
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: documentation This PR works on the website or other text documents in the repo.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

https://docusaurus.io/zh-CN//feature-requests -> 404
2 participants