-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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: Upgrading api to use drf 5th api. register_and_enroll_students api #35084
Conversation
b21a992
to
9363631
Compare
9363631
to
839406e
Compare
@feanil Please review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to be merged.
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
2U Release Notice: This PR has been deployed to the edX production environment. |
1 similar comment
2U Release Notice: This PR has been deployed to the edX production environment. |
…api (openedx#35084) * feat: upgrading simple api to drf compatible.
Issue for tracking.
Updating api code using DRF.
Steps to verify
ALLOW_AUTOMATED_SIGNUPS
set this to True inlms/djangoapps/instructor/settings/common.py