-
Notifications
You must be signed in to change notification settings - Fork 29
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
I18N fixes: Fallback translations #52
Conversation
✅ Deploy Preview for nimble-elf-d9d491 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
"vitepress": "^1.0.0-rc.33", | ||
"vitepress-versioning-plugin": "^0.10.0", | ||
"vue": "^3.4.0" | ||
"patch-package": "^8.0.0", |
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.
is this not a devDependencies?
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.
Potentially? I didn't test if it worked as a devDependency
Uses a patch to apply vuejs/vitepress#3312 to vitepress, this should work on all future vitepress versions (until 1.0.0, but by then, the PR is likely merged) - this was recommended by the vitepress maintainer as a workaround solution that will work in the long term.
Also fixes URLs on the sidebar by prepending the locale URL.