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

[2.1.0] Merge translations #6148

Merged
merged 20 commits into from
Oct 19, 2021
Merged

[2.1.0] Merge translations #6148

merged 20 commits into from
Oct 19, 2021

Conversation

conorsch
Copy link
Contributor

Status

Ready for review

Description of Changes

Merge translations for 2.1.0. See docs at https://docs.securedrop.org/en/stable/development/i18n.html#release-day

Testing

  • CI must be green, with translation tests run.

@conorsch conorsch requested a review from a team as a code owner October 19, 2021 15:44
@conorsch
Copy link
Contributor Author

CI failed: https://app.circleci.com/pipelines/github/freedomofpress/securedrop/3133/workflows/98b1e555-f97f-45f9-8b00-40a7affe4e4f/jobs/57441 with a bunch of selenium timeouts. Naively re-running CI, will investigate more closely if it fails again.

@conorsch
Copy link
Contributor Author

Failed again: https://app.circleci.com/pipelines/github/freedomofpress/securedrop/3133/workflows/2991c4e8-d67a-4a13-a8d3-b7e2cbaf0cba/jobs/57442 with messages like

 <class 'selenium.common.exceptions.ElementNotInteractableException'>
	Message: Element <a id="login-button" class="btn secondary" href="/login"> could not be scrolled into view

@cfm, mind taking a look here? I'm not sure we've run the translation tests successfully against the accessibility changes yet.

@conorsch
Copy link
Contributor Author

I'm not sure we've run the translation tests successfully against the accessibility changes yet.

If I'm reading the history correctly, we coulda/shoulda requested that #6041 be submitted from a branch named i18n-* to catch these failures earlier. Building these changes locally and spot-checking a few languages, I can't identify any problems. Does it make sense to you that the aria-labels would break the selen tests? Maybe I haven't identified the right language locally to check yet....

Conor Schaefer added 20 commits October 19, 2021 10:42
contributors:
  Cory Francis Myers
  Erik Moeller

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

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  commit: d7dd943
contributors:
  michaela-bot

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  commit: d7dd943
contributors:
  Adrian
  Dimitris Maroulidis

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  commit: d7dd943
contributors:
  Zuhualime Akoochimoya

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  commit: d7dd943
contributors:
  AO Localization Lab
  Erik Moeller

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  commit: d7dd943
contributors:
  Cory Francis Myers
  Erik Moeller

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  commit: d7dd943
contributors:
  Oktavia
  Sveinn í Felli

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  commit: d7dd943
contributors:
  Claudio Arseni
  bovirus

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  commit: d7dd943
contributors:
  Øyvind Bye Skille

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  commit: d7dd943
contributors:
  Cory Francis Myers
  Erik Moeller

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  commit: d7dd943
contributors:
  Cory Francis Myers
  Erik Moeller

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  commit: d7dd943
contributors:
  Cory Francis Myers
  Erik Moeller

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  commit: d7dd943
contributors:
  Cory Francis Myers
  Erik Moeller

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  commit: d7dd943
contributors:
  1000101
  Katarina Kasalova

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  commit: d7dd943
contributors:
  Chi-Hsun Tsai
  mengpangwang

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  commit: d7dd943
@cfm
Copy link
Member

cfm commented Oct 19, 2021

In investigating with @conorsch, we discovered that a string in fr_FR was missing a closing (escaped) quotation mark, breaking in that locale:

  1. the DOM on any page that includes this string;
  2. any of the {source,journalist}_navigation_steps that visits such a page; and thus
  3. any test that includes such a step.

This is fixed in freedomofpres/securedrop-i18n@d7dd943 and included under test here in 4baf9eb#diff-6875d0f1666d8d0824d09e3e624b6f175ce1fd4bffe415a73b59af8f18b2a9ddR745.

@conorsch
Copy link
Contributor Author

Aye, CI is passing! Ready for final review. After merge, we'll still need to backport into the release branch.

Copy link
Member

@cfm cfm left a comment

Choose a reason for hiding this comment

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

Approved per checklist:

  • CI must be green (at last!), with translation tests run.

@cfm cfm merged commit 3182789 into develop Oct 19, 2021
conorsch added a commit that referenced this pull request Oct 19, 2021
[2.1.0]Backport #6148 (merge translations)
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