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

Force translation update when generating new bundles #1516

Merged
merged 2 commits into from
Oct 30, 2019

Conversation

koke
Copy link
Member

@koke koke commented Oct 30, 2019

While testing #1503 I noticed some translations missing. The current yarn i18n-cache is overly lazy when it comes to downloading translations, and it will only update them if they have never been cached before.

This might be ok during development, but we need to ensure our releases always get the right translations. IMO, this includes any betas or intermediate builds we produce.

For this, I added a prebundle script that runs the new i18n-cache:force script, and will download or update any existing translations.

To test:

  • Run yarn bundle
  • Verify that translations are always downloaded

Update release notes:

  • If there are user facing changes, I have added an item to RELEASE-NOTES.txt.

@koke koke added this to the 1.17 milestone Oct 30, 2019
@koke koke requested review from Tug and mchowning October 30, 2019 10:23
Copy link
Contributor

@hypest hypest left a comment

Choose a reason for hiding this comment

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

This makes sense, LGTM!

@koke koke merged commit e776a6c into develop Oct 30, 2019
@koke koke deleted the force-i18n-cache-on-bundle branch October 30, 2019 10:55
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.

2 participants