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

feat : load translations from icm #695

Merged
merged 21 commits into from
Jun 1, 2021
Merged

Conversation

MaxKless
Copy link
Collaborator

@MaxKless MaxKless commented May 12, 2021

PR Type

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] CI-related changes
[ ] Documentation content changes
[ ] Application / infrastructure changes
[ ] Other:

What Is the Current Behavior?

Currently, it is only possible to change translations by updating one of the translation files and re-deploying the PWA.

Issue Number: Closes #

With this PR, it will be possible to update translations in the ICM or somewhere remote and load them into the PWA without redeploying.

ToDo

  • refine process
  • Update the loader for localization api
  • fix xliff conversion script for object
  • ensure SSR functionality
  • fix all e2e tests
  • docs

Does this PR Introduce a Breaking Change?

[ ] Yes
[ ] No

Other Information

@MaxKless MaxKless added the enhancement Enhancement to an existing feature label May 12, 2021
@MaxKless MaxKless force-pushed the feat/load-translations-from-icm branch from 92a5d6c to 490dd8c Compare May 18, 2021 16:34
@MaxKless MaxKless added this to the 0.31 milestone May 19, 2021
@MaxKless MaxKless force-pushed the feat/load-translations-from-icm branch 3 times, most recently from 7744ef3 to 6adcb38 Compare May 20, 2021 13:43
@jometzner jometzner self-assigned this May 25, 2021
@jometzner
Copy link
Collaborator

Cypres test is failing due to issue with ICM localization resource (IS-32998).

@jometzner jometzner self-requested a review May 26, 2021 11:20
@jometzner jometzner force-pushed the feat/load-translations-from-icm branch 2 times, most recently from 72d73cd to c7b9250 Compare May 26, 2021 15:28
@jometzner jometzner requested a review from marschmidt89 May 27, 2021 14:14
marschmidt89
marschmidt89 previously approved these changes May 28, 2021
Copy link
Contributor

@marschmidt89 marschmidt89 left a comment

Choose a reason for hiding this comment

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

documentation reviewed.

@jometzner jometzner force-pushed the feat/load-translations-from-icm branch from f759b6d to 533593e Compare May 31, 2021 16:09
@jometzner jometzner assigned jometzner and unassigned jometzner Jun 1, 2021
@jometzner jometzner removed their assignment Jun 1, 2021
@MaxKless MaxKless marked this pull request as ready for review June 1, 2021 13:17
@MaxKless MaxKless merged commit 7f8ae68 into develop Jun 1, 2021
@MaxKless MaxKless deleted the feat/load-translations-from-icm branch June 1, 2021 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants