-
Notifications
You must be signed in to change notification settings - Fork 675
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
Feat/i18n #607
Feat/i18n #607
Conversation
✅ Deploy Preview for tauri ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
-- Don't merge -- |
Here are the test deploy logs showing successful authentication and dry runs for crowding: https://app.netlify.com/sites/tauri/deploys/6299ef3b62286600082b039b Going to change the netlify config to now only run this in the production context |
Awesome! Do we want to merge it right or would that send the wrong message? |
imo we should merge it relatively asap, but without advertising it (for now). Just so that we can make sure everything works |
It's good to merge now. |
Now ready to turn on the fancy i18n (insert random emoji here).
Note: This is currently not going to upload any of the files in the
docs
folder; it will only be some default Docusaurus strings and anything in thepages
folder that properly has the translation elements woven in. Additional docs will be enabled for translation in their respective PRs as I'd like to use this to only enable i18n, not necessarily populate everything at once.Deploy logs of a dry-run test on this branch: https://app.netlify.com/sites/tauri/deploys/6299ef3b62286600082b039b