8000 [Translation] [Crowdin] Add a tip about language mapping by andrii-bodnar · Pull Request #19325 · symfony/symfony-docs · GitHub
[go: up one dir, main page]

Skip to content

[Translation] [Crowdin] Add a tip about language mapping #19325

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

Merged
merged 1 commit into from
Dec 26, 2023

Conversation

andrii-bodnar
Copy link
Contributor

Related PR - symfony/symfony#53178

@carsonbot carsonbot added this to the 6.4 milestone Dec 22, 2023
@OskarStark OskarStark changed the title [Translation] Add a tip about Crowdin language mapping [Translation][Crowdin] Add a tip about language mapping Dec 22, 2023
@OskarStark OskarStark added the Waiting Code Merge Docs for features pending to be merged label Dec 22, 2023
@carsonbot carsonbot changed the title [Translation][Crowdin] Add a tip about language mapping [Translation] [Crowdin] Add a tip about language mapping Dec 22, 2023
@carsonbot carsonbot modified the milestones: 6.4, next Dec 22, 2023
@andrii-bodnar andrii-bodnar force-pushed the crowdin-language-mapping-tip branch from 01aeb67 to 0aec4ed Compare December 22, 2023 15:55
nicolas-grekas added a commit to symfony/symfony that referenced this pull request Dec 26, 2023
…i-bodnar)

This PR was submitted for the 6.4 branch but it was merged into the 6.3 branch instead.

Discussion
----------

[Translation][Crowdin] Use project language mapping

| Q             | A
| ------------- | ---
| Branch?       | 6.3
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | -
| License       | MIT

Sometimes the locale codes may be different from the [codes](https://developer.crowdin.com/language-codes/) used in the Crowdin API. This will cause API errors because the wrong code is passed in the upload and download translation requests. There was already a fix (#50040), but it only works partially.

The suggested approach uses the [language mapping](https://support.crowdin.com/project-settings/#languages) configured in the Crowdin project. If the locale code and the crowdin code are different, the user needs to configure the language mapping in the Crowdin project.

Documentation - symfony/symfony-docs#19325

Commits
-------

a1d2f67 [Translation] Crowdin Bridge: use the project language mapping
@nicolas-grekas
Copy link
Member

Could target 6.3 since I've merged symfony/symfony#53178 on 6.3

@andrii-bodnar andrii-bodnar changed the base branch from 6.4 to 6.3 December 26, 2023 14:59
@andrii-bodnar andrii-bodnar changed the base branch from 6.3 to 6.4 December 26, 2023 14:59
@andrii-bodnar
Copy link
Contributor Author

@nicolas-grekas thank you, I'll rebase

@andrii-bodnar andrii-bodnar force-pushed the crowdin-language-mapping-tip branch from 0aec4ed to e9c44f4 Compare December 26, 2023 15:10
@andrii-bodnar andrii-bodnar changed the base branch from 6.4 to 6.3 December 26, 2023 15:10
@xabbuh xabbuh removed the Waiting Code Merge Docs for features pending to be merged label Dec 26, 2023
@xabbuh xabbuh modified the milestones: next, 6.3 Dec 26, 2023
@OskarStark
Copy link
Contributor

Thank you Andrii.

@OskarStark OskarStark merged commit 92c1397 into symfony:6.3 Dec 26, 2023
@andrii-bodnar andrii-bodnar deleted the crowdin-language-mapping-tip branch December 27, 2023 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants
0