Merge pull request #233 from appsembler/bugfix/batch-enrollment-reason

Check for presence of batchEnroll.reason_field
appsembler · Mar 15, 2023 · d573783 · d573783
2 parents 5b8cc2e + ddfada1
commit d573783
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/lms/static/js/instructor_dashboard/membership.js b/lms/static/js/instructor_dashboard/membership.js
@@ -603,7 +603,7 @@ such that the value can be defined later than this assignment (file load order).
             this.$request_response_error = this.$container.find('.request-response-error');
             this.$enrollment_button.click(function(event) {
                 var sendData;
-                if (!batchEnroll.$reason_field.val()) {
+                if (batchEnroll.$reason_field.length && !batchEnroll.$reason_field.val()) {
                     batchEnroll.fail_with_error(gettext('Reason field should not be left blank.'));
                     return false;
                 }