From a9e8bd5558942761dbbd7536af7d1373f48562c2 Mon Sep 17 00:00:00 2001
From: Kyr <40792129+khudym@users.noreply.github.com>
Date: Fri, 16 Aug 2024 17:19:13 +0300
Subject: [PATCH] fix: license widget checkbox and link (#486)

* fix: share alike after save, license link for creative common

* test: update snapshot

---------

Co-authored-by: Kyrylo Hudym-Levkovych <kyr.hudym@kyrs-MacBook-Pro.local>
---
 .../LicenseWidget/LicenseDetails.jsx          |  2 +-
 .../LicenseWidget/LicenseDisplay.jsx          | 16 +++++----
 .../LicenseDisplay.test.jsx.snap              | 33 -------------------
 3 files changed, 10 insertions(+), 41 deletions(-)

diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDetails.jsx b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDetails.jsx
index cba3e3b3a..9cd415a57 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDetails.jsx
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDetails.jsx
@@ -123,7 +123,7 @@ export const LicenseDetails = ({
                     </Form.Label>
                     <ActionRow.Spacer />
                     <CheckboxControl
-                      cchecked={details.shareAlike}
+                      checked={details.shareAlike}
                       disabled={level === LicenseLevel.course}
                       onChange={(e) => updateField({
                         licenseDetails: {
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 435d9fd42..67a873cf0 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDisplay.jsx
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDisplay.jsx
@@ -28,13 +28,15 @@ export const LicenseDisplay = ({
           <LicenseBlurb license={license} details={details} />
           <div className="x-small mt-3">{licenseDescription}</div>
         </div>
-        <Hyperlink
-          className="text-primary-500 x-small"
-          destination="https://creativecommons.org/about"
-          target="_blank"
-        >
-          <FormattedMessage {...messages.viewLicenseDetailsLabel} />
-        </Hyperlink>
+        {license === LicenseTypes.creativeCommons && (
+          <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 488422c90..655802d54 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,17 +26,6 @@ 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>
 `;
 
@@ -66,17 +55,6 @@ exports[`LicenseDisplay snapshots snapshots: renders as expected with level set
       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>
 `;
 
@@ -148,16 +126,5 @@ exports[`LicenseDisplay snapshots snapshots: renders as expected with level set
       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>
 `;