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

Missing strings for Javascript Translations #1447

Closed
wants to merge 4 commits into from

Conversation

bartdelvaux
Copy link
Contributor

The strings of static html files (like Magento_Checkout/web/template/minicart/content.html) are not loaded in the js-translation.json even if there are translations.

When looking in the code, the search for strings in these files are not loaded.

With this fix, these files will be in the list and could be translated trough the jstranslator function

…loaded in the js-translate.json

With this fix, these files will be in the list and could be translated trough the jstranslator function
@vpelipenko vpelipenko added the PS label Jul 7, 2015
@vrann vrann added the Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development label Jul 8, 2015
@vrann
Copy link
Contributor

vrann commented Jul 8, 2015

@bartdelvauxx Thanks for PR, changes looks good. Can you please make sure all Travis builds passed.

@vpelipenko
Copy link
Contributor

Failed integration tests is known issue and we are working on it, but unit tests should be fixed.

@vpelipenko
Copy link
Contributor

@bartdelvauxx, thank you again for contribution. This PR was accepted and merged, but wasn't closed automatically by GitHub.

@vpelipenko vpelipenko closed this Jul 21, 2015
magento-team pushed a commit that referenced this pull request Sep 1, 2017
[EngCom] Public Pull Requests - 2.1
 - MAGETWO-72135: Bugfix - Multiple filter_url_params #10714
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development Progress: accept
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants