feat: add display on org page field in AdditionalMetadata (#891)

* feat: add display on org page field in AdditionalMetadata
openedx · Jul 25, 2023 · 8ab3e35 · 8ab3e35
1 parent 80b4a1e
commit 8ab3e35
Show file tree

Hide file tree

Showing 5 changed files with 67 additions and 3 deletions.
diff --git a/src/components/EditCoursePage/AdditionalMetadataFields.jsx b/src/components/EditCoursePage/AdditionalMetadataFields.jsx
@@ -41,6 +41,19 @@ const AdditionalMetadataFields = (props) => {
     <div className="collapsible-card pgn_collapsible mt-4">
       <div className="collapsible-body">
         <FieldLabel text="Additional Metadata Details" className="h3 font-weight-normal mb-3" />
+        <div className="mb-3 mt-1">
+          <Field
+            name="additional_metadata.display_on_org_page"
+            type="checkbox"
+            label="additional_metadata.display_on_org_page"
+            disabled={disabled}
+            component="input"
+            required={isRequiredField('display_on_org_page')}
+            className="float-left mt-1 ml-1"
+          />
+          <FieldLabel text="Display on Organization Page" className="font-weight-normal ml-2 float-left" />
+          <br />
+        </div>
         <Field
           name="additional_metadata.external_url"
           component={RenderInputTextField}
@@ -51,7 +64,7 @@ const AdditionalMetadataFields = (props) => {
         <Field
           name="additional_metadata.external_identifier"
           component={RenderInputTextField}
-          label=<FieldLabel id="external_identifier.label" text="2U system identifier" />
+          label=<FieldLabel id="external_identifier.label" text="External system identifier" />
           disabled={disabled}
           required={isRequiredField('external_identifier')}
         />

diff --git a/src/components/EditCoursePage/EditCoursePage.test.jsx b/src/components/EditCoursePage/EditCoursePage.test.jsx
@@ -928,6 +928,7 @@ describe('EditCoursePage', () => {
         external_identifier: '2U_external_identifier',
         lead_capture_form_url: 'https://www.lead_capture_url.com',
         organic_url: 'https://www.organic_url.com',
+        display_on_org_page: true,
         certificate_info: {
           heading: 'heading',
           blurb: 'blurb',

diff --git a/src/components/EditCoursePage/__snapshots__/AdditionalMetadataFields.test.jsx.snap b/src/components/EditCoursePage/__snapshots__/AdditionalMetadataFields.test.jsx.snap
@@ -15,6 +15,28 @@ exports[`AdditionalMetadata Fields Display all fields 1`] = `
       optional={false}
       text="Additional Metadata Details"
     />
+    <div
+      className="mb-3 mt-1"
+    >
+      <Field
+        className="float-left mt-1 ml-1"
+        component="input"
+        disabled={false}
+        label="additional_metadata.display_on_org_page"
+        name="additional_metadata.display_on_org_page"
+        required={true}
+        type="checkbox"
+      />
+      <FieldLabel
+        className="font-weight-normal ml-2 float-left"
+        extraText=""
+        helpText=""
+        id={null}
+        optional={false}
+        text="Display on Organization Page"
+      />
+      <br />
+    </div>
     <Field
       component={[Function]}
       disabled={false}
@@ -41,7 +63,7 @@ exports[`AdditionalMetadata Fields Display all fields 1`] = `
           helpText=""
           id="external_identifier.label"
           optional={false}
-          text="2U system identifier"
+          text="External system identifier"
         />
       }
       name="additional_metadata.external_identifier"
@@ -430,6 +452,28 @@ exports[`AdditionalMetadata Fields Display required fields on the basis of exter
       optional={false}
       text="Additional Metadata Details"
     />
+    <div
+      className="mb-3 mt-1"
+    >
+      <Field
+        className="float-left mt-1 ml-1"
+        component="input"
+        disabled={false}
+        label="additional_metadata.display_on_org_page"
+        name="additional_metadata.display_on_org_page"
+        required={true}
+        type="checkbox"
+      />
+      <FieldLabel
+        className="font-weight-normal ml-2 float-left"
+        extraText=""
+        helpText=""
+        id={null}
+        optional={false}
+        text="Display on Organization Page"
+      />
+      <br />
+    </div>
     <Field
       component={[Function]}
       disabled={false}
@@ -456,7 +500,7 @@ exports[`AdditionalMetadata Fields Display required fields on the basis of exter
           helpText=""
           id="external_identifier.label"
           optional={false}
-          text="2U system identifier"
+          text="External system identifier"
         />
       }
       name="additional_metadata.external_identifier"

diff --git a/src/components/EditCoursePage/__snapshots__/EditCoursePage.test.jsx.snap b/src/components/EditCoursePage/__snapshots__/EditCoursePage.test.jsx.snap
@@ -574,6 +574,7 @@ exports[`EditCoursePage renders page correctly with courseInfo 1`] = `
             "certificate_info_blurb": "blurb",
             "certificate_info_heading": "heading",
             "course_term_override": undefined,
+            "display_on_org_page": undefined,
             "end_date": "2019-05-10T00:00:00Z",
             "external_course_marketing_type": "short_course",
             "external_identifier": "2U_external_identifier",
@@ -1451,6 +1452,7 @@ exports[`EditCoursePage renders page correctly with courseInfo and courseOptions
             "certificate_info_blurb": "blurb",
             "certificate_info_heading": "heading",
             "course_term_override": undefined,
+            "display_on_org_page": undefined,
             "end_date": "2019-05-10T00:00:00Z",
             "external_course_marketing_type": "short_course",
             "external_identifier": "2U_external_identifier",
@@ -2614,6 +2616,7 @@ exports[`EditCoursePage renders page correctly with courseInfo, courseOptions, a
             "certificate_info_blurb": "blurb",
             "certificate_info_heading": "heading",
             "course_term_override": undefined,
+            "display_on_org_page": undefined,
             "end_date": "2019-05-10T00:00:00Z",
             "external_course_marketing_type": "short_course",
             "external_identifier": "2U_external_identifier",

diff --git a/src/components/EditCoursePage/index.jsx b/src/components/EditCoursePage/index.jsx
@@ -277,6 +277,8 @@ class EditCoursePage extends React.Component {
       ],
       start_date: courseData.additional_metadata.start_date,
       end_date: courseData.additional_metadata.end_date,
+      // eslint-disable-next-line max-len
+      display_on_org_page: courseData.additional_metadata.display_on_org_page !== undefined ? courseData.additional_metadata.display_on_org_page : true,
       product_status: courseData.additional_metadata.product_status,
       external_course_marketing_type: courseData.additional_metadata.external_course_marketing_type,
       product_meta: productMeta ? {
@@ -490,6 +492,7 @@ class EditCoursePage extends React.Component {
         facts_2_blurb: additional_metadata.facts[1]?.blurb,
         start_date: additional_metadata.start_date,
         end_date: additional_metadata.end_date,
+        display_on_org_page: additional_metadata.display_on_org_page,
         external_course_marketing_type: additional_metadata.external_course_marketing_type,
         product_status: additional_metadata.product_status,
         product_meta_title: additional_metadata.product_meta?.title,