Skip to content

[Not to merge] New Crowdin updates#810

Closed
veloce wants to merge 1005 commits into
mainfrom
l10n_main
Closed

[Not to merge] New Crowdin updates#810
veloce wants to merge 1005 commits into
mainfrom
l10n_main

Conversation

@veloce

@veloce veloce commented Jul 1, 2024

Copy link
Copy Markdown
Contributor

Automatic PR from crowdin that is updated with each new translations, we don't need to merge it (and shouldn't) because we must use our script to integrate new translations.

@olivertzeng

Copy link
Copy Markdown

Stupid question but will this be merged?

@veloce

veloce commented Sep 14, 2024

Copy link
Copy Markdown
Contributor Author

No, it is an automatic PR from crowdin but we just won't merge it. The translations are integrated differently in this project.

@olivertzeng

Copy link
Copy Markdown

No, it is an automatic PR from crowdin but we just won't merge it. The translations are integrated differently in this project.

So if I were to submit translations I should be on crowdin instead of having a PR to the i18n project?

@olivertzeng

Copy link
Copy Markdown

@veloce After a brief view all zh-TW xml strings are correct but some are not in .arb files, should i change them?

@veloce veloce marked this pull request as draft June 3, 2025 17:25
@veloce veloce changed the title New Crowdin updates [Not to merge] New Crowdin updates Jun 3, 2025
@veloce veloce closed this Jun 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants