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

Update translations for 1.0.0 #4806

Merged
merged 17 commits into from
Sep 16, 2019
Merged

Conversation

rmol
Copy link
Contributor

@rmol rmol commented Sep 12, 2019

Status

Ready for review

Description of Changes

Updated translations for 1.0.0.

Testing

  • Check out this branch.
  • Run the page layout tests: make test TESTFILES=tests/pageslayout
  • Check some of the screenshots under securedrop/tests/pageslayout/screenshots/
  • Run the development server and check the strings changed for 1.0.0 by visiting with a couple of languages and verifying that the strings are in fact translated. The source start page has several, e.g. "First submission" and "Return visit".

Deployment

No deployment concerns.

Checklist

If you made non-trivial code changes:

  • I have written a test plan and validated it for this PR

@rmol rmol changed the title I18n merge 1.0.0 Update translations for 1.0.0 Sep 12, 2019
@conorsch
Copy link
Contributor

Most recent build shows CI failure as described in #4818. Since #4819 landed on develop, a rebase on this branch should resolve, getting the translation tests passing reliably again.

@rmol rmol closed this Sep 16, 2019
@rmol rmol deleted the i18n-merge-1.0.0 branch September 16, 2019 17:18
@rmol rmol restored the i18n-merge-1.0.0 branch September 16, 2019 17:24
@rmol rmol reopened this Sep 16, 2019
rmol added 17 commits September 16, 2019 13:26
contributors:
  Thalia Rahme

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  revision: 75d090c
contributors:
  Benet (BennyBeat) R. i Camps
  John Smith

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  revision: 75d090c
contributors:
  Ettore Atalan
  John Hensley
  Robin Schubert

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  revision: 75d090c
contributors:
  Dimitris Maroulidis

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  revision: 75d090c
contributors:
  Adolfo Jayme-Barrientos
  carlos
  erinm

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  revision: 75d090c
contributors:
  AbhayKaushik
  Chandan Kumar (raukadah)
  Drashti

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  revision: 75d090c
contributors:
  Oktavia
  Sveinn í Felli

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  revision: 75d090c
contributors:
  Beatrice Martini
  Claudio Arseni
  Giandomenico Lombardi

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  revision: 75d090c
contributors:
  John Hensley
  Øyvind Bye Skille

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  revision: 75d090c
contributors:
  Pander
  Thom
  kwadronaut

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  revision: 75d090c
contributors:
  mickael e

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  revision: 75d090c
contributors:
  Adham Kurbanov
  erinm

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  revision: 75d090c
contributors:
  Jonas Franzén

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  revision: 75d090c
contributors:
  Kaya Zeren
  tekrei

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  revision: 75d090c
@rmol rmol marked this pull request as ready for review September 16, 2019 18:40
Copy link
Contributor

@emkll emkll left a comment

Choose a reason for hiding this comment

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

Thanks @rmol ! Reviewed the diff for all language and ensured no tags/scripts were introduced in the translations, LGTM. Small note re: incompletely translated language, let's observe the behavior in RC4.

Good to merge once the translation tests pass

@@ -983,6 +983,16 @@ msgstr ""
msgid "Forgot your codename?"
msgstr "Ți-ai uitat numele de cod?"

#: source_templates/footer.html:7
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's make sure we test the footer behavior with ro and ensure the english-language messages are properly displayed for these strings.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

They are.

romanian

@emkll
Copy link
Contributor

emkll commented Sep 16, 2019

All translation tests passed in https://circleci.com/gh/freedomofpress/securedrop/34351 , merging!

@emkll emkll merged commit 23c4b6c into freedomofpress:develop Sep 16, 2019
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.

3 participants