Repository navigation
feat: add instructor task for async batch enrollment (#37216) - #91
Merged
Merged
Conversation
johanseto
approved these changes
Aug 4, 2026
johanseto
left a comment
johanseto
left a comment
There was a problem hiding this comment.
I was looking the way to test, however as its a backport lets continue =)
andrey-canon
changed the base branch from
and/FUTUREX-1331
to
open-release/verawood.nelp
August 4, 2026 16:55
Author
The link for the original pr is in the description, there you can find how to test this |
* feat: add instructor task for async batch enrollment * feat: add tests for submit_student_enrollment_batch api function * feat: add tests for async enrollment and unenrollment processes * feat: add unit tests for student enrollment utility functions * refactor: update user identification method * test: uncomment tests * fix: satisfy ruff and pylint rules * fix: correct state transition * fix: add type hints for request_user parameter and improve error handling * fix: improve error messages and task key generation in enrollment processes * chore: implement suggestions from code review * fix: update task key generation in tests * fix: update state transition in bulk enrollment tests * fix: disable pylint warning for imported User model in enrollment.py * fix: correct test method names and update state transitions in bulk enrollment tests * fix: wrap enrollment sync process in a transaction to ensure atomicity * feat: add site_id parameter to enrollment batch processing * fix: add site_id to the enrollment batch submission test case * fix: wrap enrollment and audit operations in a transaction for atomicity
andrey-canon
force-pushed
the
and/FUTUREX-1792
branch
from
August 4, 2026 17:41
afe6871 to
7345b0f
Compare
andrey-canon
temporarily deployed
to
open-release/verawood.nelp
August 5, 2026 20:44 — with 
GitHub Actions
Inactive
GitHub Actions
Inactive
andrey-canon
deployed
to
open-release/verawood.nelp
August 5, 2026 20:44 — with
GitHub Actions
Active
GitHub Actions
Active
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This is a backport of openedx#37216
Issue # 1792
Migration pr of #79