Skip to content

CMS i18n changes from Tsinghua - #443

Merged
cpennington merged 2 commits into
masterfrom
tsinghuax/cms-i18n-leftovers
Jul 22, 2013
Merged

CMS i18n changes from Tsinghua#443
cpennington merged 2 commits into
masterfrom
tsinghuax/cms-i18n-leftovers

Conversation

@cpennington

Copy link
Copy Markdown
Contributor

@singingwolfboy these changes were left over after you landed the cms i18n changes. Can you review them?

Comment thread cms/static/js/base.js Outdated

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.

We're going to loose a space here. Seems like we should add a   right before the . I'll do it..

@chrisndodge

Copy link
Copy Markdown
Contributor

Also, need to chase down why those two files are being deleted. Doesn't seem related to I18N....

@chrisndodge

Copy link
Copy Markdown
Contributor

I think those deleted files are OK to remove

+1 from me. I don't know if @singingwolfboy needs to review as well.

@singingwolfboy

Copy link
Copy Markdown
Contributor

Looks good to me. Those files were deleted intentionally -- they aren't used anywhere.

cpennington added a commit that referenced this pull request Jul 22, 2013
@cpennington
cpennington merged commit cba9827 into master Jul 22, 2013
@cpennington
cpennington deleted the tsinghuax/cms-i18n-leftovers branch July 22, 2013 16:21
chrisrossi pushed a commit to jazkarta/edx-platform that referenced this pull request Mar 31, 2014
e-kolpakov referenced this pull request in open-craft/openedx-platform May 15, 2015
Merge pull request #440 from edx-solutions/ekolpakov/problem-builder-mig...
hachiyanagi-ks added a commit to nttks/edx-platform that referenced this pull request Oct 21, 2015
…ange-logger-settings

Change logger settings of django.request openedx#427
diegomillan pushed a commit to eduNEXT/edx-platform that referenced this pull request Sep 14, 2016
yoann-mroz pushed a commit to weuplearning/edx-platform that referenced this pull request Nov 30, 2020
andrey-canon pushed a commit to eduNEXT/edx-platform that referenced this pull request Jan 19, 2021
Sujeet1379 pushed a commit to chandrudev/edx-platform that referenced this pull request Nov 17, 2022
Danyal-Faheem pushed a commit to Danyal-Faheem/edx-platform that referenced this pull request Jul 15, 2025
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.

3 participants