Skip to content

Commit

Permalink
docs: Crowdin + MT issue (facebook#9620)
Browse files Browse the repository at this point in the history
Co-authored-by: Dominique Eav <[email protected]>
  • Loading branch information
domeav and Dominique Eav authored Dec 8, 2023
1 parent d3752d2 commit 61116e2
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
3 changes: 3 additions & 0 deletions project-words.txt
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ Agan
alexbdebrie
Alexey
algoliasearch
Allez
Anshul
anshul
août
Expand Down Expand Up @@ -185,6 +186,7 @@ Mdxjs
mdxjs
Meilisearch
meilisearch
merveilleuse
metadatum
metastring
metrica
Expand Down Expand Up @@ -379,6 +381,7 @@ Vieira
Viet
Vinnik
vjeux
voir
waivable
WCAG
wcag
Expand Down
6 changes: 6 additions & 0 deletions website/docs/i18n/i18n-crowdin.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -515,3 +515,9 @@ import CodeBlock from '@theme/CodeBlock';
.join('\n')}
</CodeBlock>
```
### Machine Translation (MT) issue: links/image handling
Crowdin recently rolled out some major changes to the markdown file format and now the links are treated differently than they were before. Before they were considered as tags, but now they appear as plain text. Because of these changes the plain text links are passed to the MT engine which attempts to translate the target, thus breaking the translation (for instance: this string `Allez voir [ma merveilleuse page](/ma-merveilleuse-page)` is translated `Check out [my wonderful page](/my-wonderful-page)`, and this breaks docusaurus i18n workflow as the page name should not be translated).
As of 2023 Dec.7, they are not planning to change the logic of how links are treated, so you should have this in mind if you plan to use Crowdin with MT.

0 comments on commit 61116e2

Please sign in to comment.