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

Introduced a global translation helper #700

Merged
merged 13 commits into from
Jun 24, 2019
Merged

Introduced a global translation helper #700

merged 13 commits into from
Jun 24, 2019

Conversation

sznowicki
Copy link
Contributor

Description

@shopgate/engage/core exports i18n module which is pure js implementation of theme translations

Type of change

Please add an "x" into the option that is relevant:

  • Bug Fix 🐛 (non-breaking change which fixes an issue)
  • Enhancement 🚀 (non-breaking change which adds functionality)
  • Breaking Change 💥 (fix or feature that would cause existing functionality to not work as expected)
  • Polish 💅 (Just some cleanups)
  • Internal 🏠 Only relates to internal processes.

How to test it

There is an example extension created for showcase purposes which contains first implementation of the feature.
https://github.com/shopgate-professional-services/engage-translations-demo

@sznowicki sznowicki added the enhancement New feature or request label Jun 14, 2019
@sznowicki sznowicki self-assigned this Jun 14, 2019
@sznowicki sznowicki changed the title @shopgate/engage/core exports i18n module Added i18n module Jun 14, 2019
@sznowicki sznowicki marked this pull request as ready for review June 14, 2019 15:29
libraries/engage/core/helpers/i18n.js Outdated Show resolved Hide resolved
libraries/engage/core/helpers/i18n.js Show resolved Hide resolved
libraries/engage/core/helpers/i18n.js Show resolved Hide resolved
libraries/engage/core/helpers/i18n.spec.js Show resolved Hide resolved
themes/theme-gmd/index.jsx Show resolved Hide resolved
themes/theme-gmd/index.jsx Show resolved Hide resolved
@SG-Noxoreos SG-Noxoreos changed the title Added i18n module Introduced a global translation helper Jun 24, 2019
@SG-Noxoreos
Copy link
Contributor

Updated feature branch with v6.7.0 and resolved a merge conflict

@SG-Noxoreos SG-Noxoreos self-requested a review June 24, 2019 11:21
@SG-Noxoreos SG-Noxoreos merged commit 320aef4 into v6.7.0 Jun 24, 2019
@SG-Noxoreos SG-Noxoreos deleted the PWA-2046 branch June 24, 2019 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants