Skip to content

Commit 51734b5

Browse files
authored
Allow to defer enrollment without to_run (#2554)
1 parent 2bd505d commit 51734b5

File tree

2 files changed

+2
-8
lines changed

2 files changed

+2
-8
lines changed

courses/api.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -343,7 +343,7 @@ def defer_enrollment( # noqa: C901
343343
keep_failed_enrollments=keep_failed_enrollments,
344344
mode=EDX_ENROLLMENT_AUDIT_MODE,
345345
)
346-
return downgraded_enrollments, None
346+
return first_or_none(downgraded_enrollments), None
347347

348348
if not force and not from_enrollment.active:
349349
raise ValidationError(

courses/management/commands/defer_enrollment.py

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ def add_arguments(self, parser):
3434
"--to-run",
3535
type=str,
3636
help="The 'courseware_id' value for the CourseRun that you are deferring to",
37-
required=True,
3837
)
3938
parser.add_argument(
4039
"-k",
@@ -71,16 +70,11 @@ def handle(self, *args, **options): # noqa: ARG002
7170
raise CommandError(f"Invalid enrollment deferral - {exc}") # noqa: B904, EM102
7271
except Exception as exc: # noqa: BLE001
7372
raise CommandError(f"{exc}") # noqa: B904, EM102
74-
else:
75-
if not to_enrollment:
76-
raise CommandError(
77-
f"Failed to create/update the target enrollment ({to_courseware_id})" # noqa: EM102
78-
)
7973

8074
self.stdout.write(
8175
self.style.SUCCESS(
8276
f"Deferred enrollment for user: {user}\n"
8377
f"Enrollment deactivated: {enrollment_summary(from_enrollment)}\n"
84-
f"Enrollment created/updated: {enrollment_summary(to_enrollment)}"
78+
f"Enrollment created/updated: {enrollment_summary(to_enrollment) if to_enrollment else 'N/A'}"
8579
)
8680
)

0 commit comments

Comments
 (0)