From 9214b7a0c1341959f515c2509aa6c60bc070cbc5 Mon Sep 17 00:00:00 2001
From: AfaqShuaib09
Date: Tue, 25 Jul 2023 21:40:54 +0500
Subject: [PATCH] fix: move watchers field above
---
.../EditCoursePage/EditCourseForm.jsx | 48 +++++++++----------
.../EditCourseForm.test.jsx.snap | 46 +++++++++---------
2 files changed, 47 insertions(+), 47 deletions(-)
diff --git a/src/components/EditCoursePage/EditCourseForm.jsx b/src/components/EditCoursePage/EditCourseForm.jsx
index 49c57e8c7..5e0279a05 100644
--- a/src/components/EditCoursePage/EditCourseForm.jsx
+++ b/src/components/EditCoursePage/EditCourseForm.jsx
@@ -372,6 +372,30 @@ export class BaseEditCourseForm extends React.Component {
disabled={disabled || !administrator}
optional
/>
+ {administrator && (
+
+ A list of email addresses that will receive
+ notifications when the course run of the course is published or reviewed.
+
+ )}
+ optional
+ />
+ )}
+ isMulti
+ disabled={!(courseInfo?.data?.course_run_statuses?.includes(REVIEW_BY_INTERNAL) && administrator)}
+ optional
+ isCreatable
+ createOptionValidator={emailValidate}
+ />
+ )}
{parsedProductSource}
@@ -1093,30 +1117,6 @@ export class BaseEditCourseForm extends React.Component {
disabled={disabled}
optional
/>
- {administrator && (
-
- A list of email addresses that will receive
- notifications when the course run of the course is published or reviewed.
-
- )}
- optional
- />
- )}
- isMulti
- disabled={!(courseInfo?.data?.course_run_statuses?.includes(REVIEW_BY_INTERNAL) && administrator)}
- optional
- isCreatable
- createOptionValidator={emailValidate}
- />
- )}
{administrator && (
<>
diff --git a/src/components/EditCoursePage/__snapshots__/EditCourseForm.test.jsx.snap b/src/components/EditCoursePage/__snapshots__/EditCourseForm.test.jsx.snap
index 5f38d0eb6..9337ca4cc 100644
--- a/src/components/EditCoursePage/__snapshots__/EditCourseForm.test.jsx.snap
+++ b/src/components/EditCoursePage/__snapshots__/EditCourseForm.test.jsx.snap
@@ -5683,6 +5683,29 @@ exports[`BaseEditCourseForm renders html correctly with administrator being true
pattern="^[a-z0-9_]+(?:-[a-z0-9_]+)*$"
type="text"
/>
+
+ A list of email addresses that will receive notifications when the course run of the course is published or reviewed.
+
+ }
+ id="watchers.label"
+ optional={true}
+ text="Watchers"
+ />
+ }
+ name="watchers_list"
+ optional={true}
+ />
-
- A list of email addresses that will receive notifications when the course run of the course is published or reviewed.
-
- }
- id="watchers.label"
- optional={true}
- text="Watchers"
- />
- }
- name="watchers_list"
- optional={true}
- />