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

I18n fixes #4972

Merged
merged 7 commits into from
Mar 9, 2016
Merged

I18n fixes #4972

merged 7 commits into from
Mar 9, 2016

Conversation

aronasorman
Copy link
Collaborator

Summary

Some i18n fixes to get ready for the 0.16 po file release.

Reviewer guidance

Review by commit. Changes will make more sense.

@aronasorman aronasorman added this to the 0.16.0 milestone Mar 8, 2016
Will be used by the create_dummy_language_pack command.
@aronasorman aronasorman force-pushed the i18n-fixes branch 3 times, most recently from efedc3b to c796d64 Compare March 8, 2016 19:38
@aronasorman
Copy link
Collaborator Author

Tests are now passing.

Extract the strings from the inline help narratives yml files. Returns an
iterator containing the po file entries. Optional inline_help_parameter
specifies where the inline help narratives path is. Else, it defaults
to settings.CONTENT_DATA_PATH + "narratives.yml"

This comment was marked as spam.

@MCGallaspy
Copy link
Contributor

Looking good, great commit history! Just one comment above on a docstring/unused parameter.

@aronasorman
Copy link
Collaborator Author

Addressed comment. Tests pass.

MCGallaspy added a commit that referenced this pull request Mar 9, 2016
@MCGallaspy MCGallaspy merged commit 93ebeb1 into learningequality:0.16.x Mar 9, 2016
@MCGallaspy MCGallaspy removed the has PR label Mar 9, 2016
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.

2 participants