Merge pull request #234 from appsembler/juniper/main

bryanlandia · web-flow · commit 2cd899e65a54 · 2023-05-15T14:50:45.000-07:00
Update from `juniper/main` (production)
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;
                 }

Original file line number	Diff line number	Diff line change
`@@ -603,7 +603,7 @@ such that the value can be defined later than this assignment (file load order).`
`603`	`603`	`this.$request_response_error = this.$container.find('.request-response-error');`
`604`	`604`	`this.$enrollment_button.click(function(event) {`
`605`	`605`	`var sendData;`
`606`		`- if (!batchEnroll.$reason_field.val()) {`
	`606`	`+ if (batchEnroll.$reason_field.length && !batchEnroll.$reason_field.val()) {`
`607`	`607`	`batchEnroll.fail_with_error(gettext('Reason field should not be left blank.'));`
`608`	`608`	`return false;`
`609`	`609`	`}`