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: allow translations to be loaded from an external source #927

Conversation

dhhyi
Copy link
Collaborator

@dhhyi dhhyi commented Nov 15, 2021

PR Type

[x] Feature

What Is the Current Behavior?

Translation cannot be loaded from an external source.
ICM translation management has some limitations:

  • Translations can't be shared between locales of brands (depending on the channel setup)

What Is the New Behavior?

An external translation source can be defined.

For testing:

  1. start external translation source: npx http-server src/assets/i18n --cors
  2. set translateLoaderUrl: 'http://localhost:8080' in environment.development.ts
  3. play around with translateLoaderFormat: '$language_$COUNTRY.json' in environment.development.ts

Does this PR Introduce a Breaking Change?

[ ] Yes
[ ] No

Other Information

AB#71316

@dhhyi dhhyi added the feature New feature or request label Nov 15, 2021
@shauke shauke requested review from MaxKless and jometzner November 26, 2021 07:20
@shauke shauke added this to the 2.0 milestone Nov 26, 2021
@dhhyi dhhyi added the community Community contributions label Dec 14, 2021
@MaxKless MaxKless removed this from the 2.0 milestone Jan 10, 2022
@dhhyi dhhyi force-pushed the feature/alternate-localizations-source-translate-manager branch 4 times, most recently from a5fa1d4 to ccb099c Compare May 10, 2022 14:13
@dhhyi dhhyi marked this pull request as ready for review May 10, 2022 14:15
@shauke shauke added wait Waiting for something (e.g. new ICM release) vote Open for community input (pro/contra) labels May 17, 2022
@shauke
Copy link
Collaborator

shauke commented May 17, 2022

We are interested in feedback in how many projects would want to use an external translation system instead of the localization functionality of ICM used by default.

@dhhyi dhhyi force-pushed the feature/alternate-localizations-source-translate-manager branch from ccb099c to e8185a1 Compare June 6, 2022 12:45
@shauke
Copy link
Collaborator

shauke commented Jul 11, 2022

Thank you @dhhyi for this pull request. But for limited request from other projects we will mot merge it into the standard Intershop PWA project and it needs to stay a project customization for you. Sorry for the late response.
We will add a link to this closed PR in the localization.md for projects that might be interested in your idea in the future.

@shauke shauke closed this Jul 11, 2022
shauke pushed a commit that referenced this pull request Jul 12, 2022
* docs: add reference to implementation for loading translations from an external source (see #927)

BREAKING CHANGES: The @ngx-translate/http-loader has been removed from the dependencies.
@shauke shauke deleted the feature/alternate-localizations-source-translate-manager branch January 6, 2023 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Community contributions feature New feature or request vote Open for community input (pro/contra) wait Waiting for something (e.g. new ICM release)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants