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(website): fix i18n routes for Canny board #5902
Conversation
✔️ [V2] 🔨 Explore the source changes: a735eeb 🔍 Inspect the deploy log: https://app.netlify.com/sites/docusaurus-2/deploys/6188aff49c4ea9000768f088 😎 Browse the preview: https://deploy-preview-5902--docusaurus-2.netlify.app |
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://deploy-preview-5902--docusaurus-2.netlify.app/ |
Size Change: +124 B (0%) Total Size: 881 kB
ℹ️ View Unchanged
|
exact: false, | ||
component: '@site/src/featureRequests/FeatureRequestsPage', | ||
modules: { | ||
basePath: './feature-requests-plugin/default/paths.json', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
createData
returns the module path, you don't need to hardcode it :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Haha, I never understood how these APIs work... I'll look deeper into how they're implemented and maybe improve the documentation on this😄
Motivation
Pass the base route as props so that canny can handle the route correctly.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Tested locally with
yarn start --locale zh-Hans
andhttps://localhost:3000/zh-Hans/feature-requests
is not redirected tohttps://localhost:3000/feature-requests
after Canny loads.