Skip to content

Translations for auth #771

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 18 commits into from
Jul 14, 2017
Merged

Translations for auth #771

merged 18 commits into from
Jul 14, 2017

Conversation

ashwinraghav
Copy link
Contributor

  • This change imports translations for firebase UI auth. Issues related to formatting (particularly for RTL languages) are to be expected. Fixes are welcome 👍
  • Once we see some stability with the UI for translations, we could roll forward with publishing version 11.0.1.

Change-Id: Iee013581f6d1e7e7387c06ce3895c378f03a6b23
@ashwinraghav ashwinraghav self-assigned this Jun 21, 2017
@ashwinraghav ashwinraghav requested a review from samtstern June 21, 2017 22:39
@samtstern samtstern changed the base branch from master to version-2.1.0-dev June 21, 2017 23:13
@samtstern
Copy link
Contributor

I'm gonna write some scripts to clean up these translations.

Change-Id: Iaea384e3957470181d802a702d7f1a6feffea124
Change-Id: Ibeec6996cfc24d59366cfcb94297ce9007c73b9f
@samtstern
Copy link
Contributor

The only thing we're missing now is a translation for privacy_policy.

ashwinraghav and others added 2 commits June 22, 2017 09:54
Change-Id: I2495679d27e4342946ad1744004639dc1bf43cb3
Change-Id: Ie45aa4e35e5078e8f6b08c1015f4b075c54f0f2f
samtstern and others added 5 commits June 27, 2017 11:18
Change-Id: I1ffcc0168c0ab29bbfefaaf746ead0aa4da648e0
… into translations

Change-Id: I6b3c7b6c30e4fb699eabd4e1cd019edf84046b4a
Change-Id: I530e90a032b58211ba6ee85595f7c43a0f499bdb
Change-Id: Ibb8a5b10395127e428ce21b9cd27fbd8535ed9a2
@samtstern
Copy link
Contributor

@ashwinraghav did you have a reason for disabling the MissingTranslation lint? I think we need that to make sure we have complete coverage.

Change-Id: I2a91ae6a2a8d8950713aafa2391c16e4afc1fbcd
<string name="resend_code" translation_description="Link text to resend verification sms">Code opnieuw verzenden</string>
<string name="verify_phone_number" translation_description="Button text to submit confirmation code and complete phone verification">Telefoonnummer verifiëren</string>
<string name="continue_phone_login" translation_description="Button text to submit phone number and send sms">Doorgaan</string>
<string name="sms_terms_of_service" translation_description="Fine print warning displayed below Verify Phone Number button">Als u op \'%1$s\', ontvangt u mogelijk een sms. Er kunnen sms- en datakosten in rekening worden gebracht.</string>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A verb is missing in this sentence. It should be Als u op \'%1$s\' tikt, ontvangt u mogelijk een sms. Er kunnen sms- en datakosten in rekening worden gebracht.

Copy link
Contributor

@samtstern samtstern Jul 13, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this! I submitted this back as a "suggestion" in our system.

Change-Id: I82c8ecd1da9bc1dabc60af948f14b44fd9a85426
Change-Id: I1d134d5d2c38d5e8b94d0ae33629dc1d23a88454
Change-Id: I5b94ff678ea3e9073024650dc6f9e14a5cfd45bb
Change-Id: I442e557cc99071685c6eee33ce85be1ee924d8e9
Change-Id: Ib5c391e9bb9162ba64419adb42b8fba219f6c7d7
@samtstern samtstern merged commit ac71879 into version-2.1.0-dev Jul 14, 2017
@samtstern samtstern deleted the translations branch July 18, 2017 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants