-
-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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: Crowdin + MT issue #9620
docs: Crowdin + MT issue #9620
Conversation
⚡️ Lighthouse report for the deploy preview of this PR
|
…reak docusaurus <-> Crowdin <-> MT workflow.
✅ [V2]Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
Thanks 👍 |
Hmmm, not sure if I can do something with the lint issues (one complains it does not understand the French words I've added in my example, and I can't reproduce why the other is failing). If anyone has a clue, let me know! |
@domeav If you want to fix them yourself, you can edit |
Warning about a recent change in the Crowdin tag handling that will break docusaurus <-> Crowdin <-> MT workflow.
Pre-flight checklist
Motivation
I've been bitten by a change in the Crowdin+MT behavior and I want to make sure other Docusaurus users know about it.
Test Plan
100% doc change
Test links
Deploy preview: https://deploy-preview-_____--docusaurus-2.netlify.app/
Related issues/PRs
#4052 (comment)