Skip to content

feat: upgrading students_update_enrollment api to DRF#37074

Merged
awais786 merged 11 commits into
openedx:masterfrom
hunzlahmalik:hunzlah/students_update_enrollment_view_to_drf
Jul 29, 2025
Merged

feat: upgrading students_update_enrollment api to DRF#37074
awais786 merged 11 commits into
openedx:masterfrom
hunzlahmalik:hunzlah/students_update_enrollment_view_to_drf

Conversation

@hunzlahmalik
Copy link
Copy Markdown
Contributor

@hunzlahmalik hunzlahmalik commented Jul 28, 2025

Copied branch from

Issue

Testing

  • Go to URL
  • Enter emails
  • Try to Enroll and Unenroll students

Enroll

image image

Unenroll

image image

Different Options

image image image image

@hunzlahmalik hunzlahmalik marked this pull request as ready for review July 28, 2025 13:28
@awais786 awais786 merged commit cdf5083 into openedx:master Jul 29, 2025
49 checks passed
@edx-pipeline-bot
Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

1 similar comment
@edx-pipeline-bot
Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

@edx-pipeline-bot
Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@jristau1984
Copy link
Copy Markdown
Contributor

jristau1984 commented Jul 29, 2025

Were validations done around bulk enrollment and unenrollment, and ensuring that Aspects gets accurate enrollment data? This is really really important code, and I dont want any regressions if they can be avoided...

@edx-pipeline-bot
Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

1 similar comment
@edx-pipeline-bot
Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

@hunzlahmalik
Copy link
Copy Markdown
Contributor Author

@jristau1984 Just to clarify...

The changes in the bulk enroll API only modify how it internally calls students_update_enrollment.
There were no changes to the input or response structure of students_update_enrollment itself.
The happy case for students_update_enrollment continues to be covered via the LMS frontend.
The happy case for bulk enrollment is verified through API test, and relevant screenshots have been attached in the PR description for reference.

BryanttV pushed a commit to eduNEXT/edx-platform that referenced this pull request Aug 1, 2025
* feat: upgrading students_update_enrollment api to DRF
sumair-arbisoft pushed a commit to sumair-arbisoft/edx-platform that referenced this pull request Aug 5, 2025
* feat: upgrading students_update_enrollment api to DRF
salman2013 pushed a commit to salman2013/edx-platform that referenced this pull request Sep 10, 2025
* feat: upgrading students_update_enrollment api to DRF
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.

4 participants