Skip to content

i18n (forums) - #1793

Closed
singingwolfboy wants to merge 1 commit into
masterfrom
man2life-i18n-forums
Closed

i18n (forums)#1793
singingwolfboy wants to merge 1 commit into
masterfrom
man2life-i18n-forums

Conversation

@singingwolfboy

Copy link
Copy Markdown
Contributor

#1785 with only forums changes

@gwprice

@singingwolfboy singingwolfboy mentioned this pull request Dec 6, 2013

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This needs to be refactored so the placeholder for the number of votes is included in the translation string

@gwprice

gwprice commented Dec 9, 2013

Copy link
Copy Markdown

How can we be sure that the gettext function is available for the .coffee files?

Also, there are a bunch of places where role names are output that need to have translation applied.

@singingwolfboy

Copy link
Copy Markdown
Contributor Author

@gwprice The best way to ensure that the gettext function is available would be to refactor the Javascript to use require.js, and declare a dependency on gettext. Short of that, you're just going to have to assume/expect that Django's JS i18n file has already been loaded.

@ghost ghost assigned gwprice Dec 16, 2013
@gwprice

gwprice commented Dec 18, 2013

Copy link
Copy Markdown

Forums team will be picking up this work soon, but I'm going to close this PR in the meantime.

@gwprice gwprice closed this Dec 18, 2013
@benpatterson
benpatterson deleted the man2life-i18n-forums branch January 21, 2015 13:12
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Mar 24, 2017
* Add font-style normal. openedx#1790

* Mod error message when disable to register student-self with input status. openedx#1787

* Mod biz-menu style height to min-height. openedx#1793

* Fix review(TYPO).
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