-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Reverting javascript translation refactoring which broke javascript translation #20109
Reverting javascript translation refactoring which broke javascript translation #20109
Conversation
…ile." This reverts commit 459ce25. The commit broke some javascript translations.
Hi @mfickers. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
Hi @mfickers , thanks for the contribution! I would prefer to keep the refactored approach. Maybe there is a way to fix the issue by i.e. introducing dependency that will allow the translation dictionary to be initialised early enough? |
Was the original PR that moved this code into an external file fixing a bug, or just an improvement? If it was just an improvement I'm in favor of a reverting for now and we can try the improvement again with some more testing later. |
Hi @VladimirZaets, thank you for the review. |
Hi @mfickers, thank you for your contribution! |
… javascript translation #20109
Description
This reverts commit 459ce25.
The commit broke some javascript translations, because after javascript translation initialisation was refactored into its own file, the translation dictionary was not initialised early enough.
Fixed Issues (if relevant)
Manual testing scenarios (*)
Contribution checklist (*)