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

OPENEUROPA-2147: Implementation of third approach with local import translations. #82

Merged
merged 3 commits into from
Aug 26, 2019

Conversation

sergepavle
Copy link
Member

OPENEUROPA-2147

Description

[notice] Imported translation for ewcms.
--
[notice] Imported translation for ewcms.
[notice] Translations imported: 654 added, 0 updated, 0 removed.
[notice] Message: 115 translation files imported. /654/ translations were added, /0/
translations were updated and /0/ translations were removed.

[Exec] Done in 15:33

Change log

  • Added:
  • Changed:
  • Deprecated:
  • Removed:
  • Fixed:
  • Security:

Commands

[Insert commands here]

@sergepavle sergepavle force-pushed the OPENEUROPA-2147-3 branch 2 times, most recently from 1d9e1ca to ded136e Compare August 21, 2019 07:50
src/LocalTranslationsBatcher.php Outdated Show resolved Hide resolved
src/LocalTranslationsBatcher.php Show resolved Hide resolved
src/LocalTranslationsBatcher.php Outdated Show resolved Hide resolved
@ademarco
Copy link
Member

ademarco commented Aug 22, 2019

I've tested it locally in different scenarios (clean install, update string from code and changes from ui not overridden) and it works well, it's around 1 minute in total. I'll approve after the remaining comments will be addressed.

@sergepavle sergepavle force-pushed the OPENEUROPA-2147-3 branch 2 times, most recently from 4f38ca5 to f619fb1 Compare August 22, 2019 12:16
ademarco
ademarco previously approved these changes Aug 22, 2019
src/LocalTranslationsBatcher.php Outdated Show resolved Hide resolved
src/LocalTranslationsBatcher.php Show resolved Hide resolved
@ademarco ademarco merged commit 504e558 into master Aug 26, 2019
@ademarco ademarco deleted the OPENEUROPA-2147-3 branch August 26, 2019 07:40
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.

3 participants