From 6368ccf5ac0554bc550e8e6f7769dcff54dfbb7c Mon Sep 17 00:00:00 2001
From: Kristin Aoki <42981026+KristinAoki@users.noreply.github.com>
Date: Mon, 5 Dec 2022 12:43:14 -0500
Subject: [PATCH] feat: show view license link at all levels (#157)

---
 .../LicenseWidget/LicenseDisplay.jsx          | 19 ++++++++-----------
 .../LicenseDisplay.test.jsx.snap              | 11 +++++++++++
 .../__snapshots__/index.test.jsx.snap         |  3 ---
 .../components/LicenseWidget/index.jsx        |  1 -
 4 files changed, 19 insertions(+), 15 deletions(-)

diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDisplay.jsx b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDisplay.jsx
index 33cb0b290..9fcfd067e 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDisplay.jsx
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDisplay.jsx
@@ -10,7 +10,7 @@ import {
   Hyperlink,
 } from '@edx/paragon';
 
-import { LicenseLevel, LicenseTypes } from '../../../../../../data/constants/licenses';
+import { LicenseTypes } from '../../../../../../data/constants/licenses';
 
 import LicenseBlurb from './LicenseBlurb';
 import { messages } from './messages';
@@ -19,7 +19,6 @@ export const LicenseDisplay = ({
   license,
   details,
   licenseDescription,
-  level,
 }) => {
   if (license !== LicenseTypes.select) {
     return (
@@ -29,15 +28,13 @@ export const LicenseDisplay = ({
           <LicenseBlurb license={license} details={details} />
           <div className="x-small">{licenseDescription}</div>
         </div>
-        {level !== LicenseLevel.course ? (
-          <Hyperlink
-            className="text-primary-500 x-small"
-            destination="https://creativecommons.org/about"
-            target="_blank"
-          >
-            <FormattedMessage {...messages.viewLicenseDetailsLabel} />
-          </Hyperlink>
-        ) : null }
+        <Hyperlink
+          className="text-primary-500 x-small"
+          destination="https://creativecommons.org/about"
+          target="_blank"
+        >
+          <FormattedMessage {...messages.viewLicenseDetailsLabel} />
+        </Hyperlink>
       </Stack>
     );
   }
diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/LicenseDisplay.test.jsx.snap b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/LicenseDisplay.test.jsx.snap
index 8fd096ee8..a853638d5 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/LicenseDisplay.test.jsx.snap
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/LicenseDisplay.test.jsx.snap
@@ -26,6 +26,17 @@ exports[`LicenseDisplay snapshots snapshots: renders as expected with default pr
       FormattedMessage component with license description
     </div>
   </div>
+  <Hyperlink
+    className="text-primary-500 x-small"
+    destination="https://creativecommons.org/about"
+    target="_blank"
+  >
+    <FormattedMessage
+      defaultMessage="View license details"
+      description="Label for view license details button"
+      id="authoring.videoeditor.license.viewLicenseDetailsLabel.label"
+    />
+  </Hyperlink>
 </Stack>
 `;
 
diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/index.test.jsx.snap b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/index.test.jsx.snap
index fe6ccb131..ea72233a5 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/index.test.jsx.snap
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/index.test.jsx.snap
@@ -35,7 +35,6 @@ exports[`LicenseWidget snapshots snapshots: renders as expected with default pro
     />
     <injectIntl(ShimmedIntlComponent)
       details={Object {}}
-      level="course"
       license="all-rights-reserved"
       licenseDescription={
         <FormattedMessage
@@ -99,7 +98,6 @@ exports[`LicenseWidget snapshots snapshots: renders as expected with isLibrary t
     />
     <injectIntl(ShimmedIntlComponent)
       details={Object {}}
-      level="library"
       license="all-rights-reserved"
       licenseDescription={
         <FormattedMessage
@@ -148,7 +146,6 @@ exports[`LicenseWidget snapshots snapshots: renders as expected with licenseType
     />
     <injectIntl(ShimmedIntlComponent)
       details={Object {}}
-      level="block"
       license="all-rights-reserved"
       licenseDescription={
         <FormattedMessage
diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/index.jsx b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/index.jsx
index dc46b9e91..cac939d74 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/index.jsx
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/index.jsx
@@ -63,7 +63,6 @@ export const LicenseWidget = ({
               license={license}
               details={details}
               licenseDescription={licenseDescription}
-              level={level}
             />
           </>
         ) : null }