Skip to content
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

Add translation project promotion link #12736

Merged
merged 1 commit into from
Dec 31, 2019
Merged

Add translation project promotion link #12736

merged 1 commit into from
Dec 31, 2019

Conversation

brawaru
Copy link
Contributor

@brawaru brawaru commented Dec 31, 2019

This PR adds promotional notice on appearance settings about translation project if any other locale than English is used. It allows users to learn and contribute translations to Mastodon.

Example screenshot

Step ahead, in this commit one unusual string is added — link to a guide. By default it refers to Crowdin project itself, but if any of Mastodon localization teams established their own guide, they can refer it. Or, if Crowdin supports localized domain for language, it can also be put there (e.g. https://fr.crowdin.com/...).

@brawaru
Copy link
Contributor Author

brawaru commented Dec 31, 2019

Can someone reassure me, in production it will fallback to English translations if some strings not translated, right? Because on testing instance it makes spans instead, so it worries me a little.

@Gargron
Copy link
Member

Gargron commented Dec 31, 2019

Can someone reassure me, in production it will fallback to English translations if some strings not translated, right?

Correct. The spans only appear in development mode.

This commit adds promotional notice on appearance settings about
translation project if any other locale than English is used. It
allows users to learn and contribute translations to Mastodon.

Step ahead, in this commit one unusual string is added - link to a
guide. By default it refers to Crowdin project itself, but if any of
Mastodon localization teams established their own guide, they can
refer it. Or, if Crowdin supports localized domain for language, it
can also be put there (e.g. https://fr.crowdin.com/...).
@Gargron Gargron merged commit 9edab7a into mastodon:master Dec 31, 2019
@brawaru brawaru deleted the feat/localization-promotion branch April 2, 2020 22:19
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.

None yet

2 participants