Skip to content

Add Course Modes to LMS - #676

Merged
dianakhuang merged 3 commits into
masterfrom
diana/course-modes
Aug 14, 2013
Merged

Add Course Modes to LMS#676
dianakhuang merged 3 commits into
masterfrom
diana/course-modes

Conversation

@dianakhuang

Copy link
Copy Markdown
Contributor

Related to PR #651, but not blocked by it. Adds a new djangoapp for maintaining varying course modes for courses.

@ormsbee

Comment thread lms/djangoapps/course_modes/models.py 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.

Break into two lines pls?

@ormsbee

ormsbee commented Aug 14, 2013

Copy link
Copy Markdown
Contributor

I think we might change the naming at some point, but I have no better suggestions at the moment. Is there an admin.py with this?

Otherwise, no objections.

@dianakhuang

Copy link
Copy Markdown
Contributor Author

I forgot the admin file. I checked it out manually to make sure it works. Still good?

@ormsbee

ormsbee commented Aug 14, 2013

Copy link
Copy Markdown
Contributor

Yup.

dianakhuang added a commit that referenced this pull request Aug 14, 2013
@dianakhuang
dianakhuang merged commit cc86fb3 into master Aug 14, 2013
@dianakhuang
dianakhuang deleted the diana/course-modes branch August 14, 2013 19:33
chrisrossi pushed a commit to jazkarta/edx-platform that referenced this pull request Mar 31, 2014
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Mar 30, 2016
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Mar 30, 2016
mtyaka referenced this pull request in open-craft/openedx-platform Apr 13, 2016
Sujeet1379 pushed a commit to chandrudev/edx-platform that referenced this pull request Nov 17, 2022
johanseto pushed a commit to nelc/edx-platform that referenced this pull request Jan 22, 2024
feat: generate valid passwords when using additional validators
DanielVZ96 referenced this pull request in open-craft/openedx-platform Jan 31, 2024
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