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

feat: add coursecreator option to django model #269

Open
wants to merge 1 commit into
base: open-release/koa.nelp
Choose a base branch
from

Conversation

johanseto
Copy link

@johanseto johanseto commented Aug 18, 2022

Description

This is related to the requirement of Nelp that is creating users but need to give course creator permissions.
The problem is that until the user login, the user will not appear in the course-creator table.
https://github.com/eduNEXT/edx-platform/blob/master/cms/djangoapps/contentstore/views/course.py#L1889-L1893
So this PR adds the option to add course creator directly from the admin model matching with the username of the user model.
Configuration
Remeber to configurate you have to update your cms/env/devstack.py or your custom settings.
FEATURES['ENABLE_CREATOR_GROUP'] = True.
Note: Is already in stage environment to check.
https://studio.stage.nelp.gov.sa/admin/course_creators/coursecreator/add/

Jira story

https://edunext.atlassian.net/browse/NELP-195?atlOrigin=eyJpIjoiNmFmOTc4YTk4NDczNGZiZGEwZjA2YzY5MjdlYzljNjQiLCJwIjoiaiJ9

Before

2022-08-18_18-03

After

2022-08-18_17-19
2022-08-18_18-10

@johanseto johanseto force-pushed the jlc/add-coursecreator-option-with-user-admin branch 2 times, most recently from 54d913a to 89967ce Compare August 18, 2022 22:44
With this change is possible add coursecreator without needing a login in cms for the user.
The add option is matched with the field username of the user model.
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.

1 participant