Skip to content

added some i18n - #1785

Closed
singingwolfboy wants to merge 2 commits into
masterfrom
man2life-i18n
Closed

added some i18n#1785
singingwolfboy wants to merge 2 commits into
masterfrom
man2life-i18n

Conversation

@singingwolfboy

Copy link
Copy Markdown
Contributor

Rebased version of #1481

PYushchenko and others added 2 commits November 26, 2013 13:46
fixes

reset i18n of non-django template

Some more i18n and fix in discussion

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's up with all these triple quotes?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The string was originally delimited using triple quotes (see line 344). I guess @man2life wanted to continue using the same delimiter.

@singingwolfboy singingwolfboy mentioned this pull request Nov 26, 2013
@sarina

sarina commented Nov 26, 2013

Copy link
Copy Markdown
Contributor

Given that we're trying real hard to deprecate lms/djangoapps/instructor/views/legacy.py I wonder if it's really worth doing translations for it.

@singingwolfboy singingwolfboy mentioned this pull request Nov 26, 2013
@singingwolfboy

Copy link
Copy Markdown
Contributor Author

Based on feedback from @gwprice, this pull request has been split into two -- one with the forums changes, and one without. I'm going to close this PR.

@benpatterson
benpatterson deleted the man2life-i18n branch January 21, 2015 13:13
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.

5 participants