From cd8be46584c5185969b9e88be0e27f7503ea1b2e Mon Sep 17 00:00:00 2001
From: kKaskak <117831817+kKaskak@users.noreply.github.com>
Date: Thu, 14 Dec 2023 12:33:10 +0100
Subject: [PATCH] refactor: naming changes + conditional rendering for the
 button

---
 src/routes/Board/Board.js                       | 17 +++++++++++------
 ...ModalData.js => useFetchNotificationData.js} |  4 ++--
 2 files changed, 13 insertions(+), 8 deletions(-)
 rename src/routes/Board/{useFetchModalData.js => useFetchNotificationData.js} (93%)

diff --git a/src/routes/Board/Board.js b/src/routes/Board/Board.js
index b9f6df20e..d8c152181 100644
--- a/src/routes/Board/Board.js
+++ b/src/routes/Board/Board.js
@@ -8,7 +8,7 @@ const { MainNavBars, MetaRow, ContinueWatchingItem, MetaItem, StreamingServerWar
 const useBoard = require('./useBoard');
 const useContinueWatchingPreview = require('./useContinueWatchingPreview');
 const styles = require('./styles');
-const useFetchModalData = require('./useFetchModalData');
+const useFetchNotificationData = require('./useFetchNotificationData');
 
 const THRESHOLD = 5;
 
@@ -19,8 +19,8 @@ const Board = () => {
     const [board, loadBoardRows] = useBoard();
     const boardCatalogsOffset = continueWatchingPreview.items.length > 0 ? 1 : 0;
     const scrollContainerRef = React.useRef();
-    const { notificationModalData, isModalDataLoading } = useFetchModalData();
-    const [isNotificationModalOpen, , closeNotificationModal, ] = useBinaryState(true);
+    const { notificationModalData, isModalDataLoading } = useFetchNotificationData();
+    const [isNotificationModalOpen, , closeNotificationModal] = useBinaryState(true);
     const onVisibleRangeChange = React.useCallback(() => {
         const range = getVisibleChildrenRange(scrollContainerRef.current);
         if (range === null) {
@@ -65,9 +65,14 @@ const Board = () => {
                                         null
                                 }
                             </div>
-                            <Button className={styles['action-button']}>
-                                <div className={styles['label']}>Learn more</div>
-                            </Button>
+                            {
+                                notificationModalData.addon.manifestUrl ?
+                                    <Button className={styles['action-button']} href={notificationModalData.addon.manifestUrl}>
+                                        <div className={styles['label']}>Learn more</div>
+                                    </Button>
+                                    :
+                                    null
+                            }
                         </div>
                     </ModalDialog>
                     :
diff --git a/src/routes/Board/useFetchModalData.js b/src/routes/Board/useFetchNotificationData.js
similarity index 93%
rename from src/routes/Board/useFetchModalData.js
rename to src/routes/Board/useFetchNotificationData.js
index b6f711fa1..022243a94 100644
--- a/src/routes/Board/useFetchModalData.js
+++ b/src/routes/Board/useFetchNotificationData.js
@@ -2,7 +2,7 @@
 
 const React = require('react');
 
-const useFetchModalData = () => {
+const useFetchNotificationData = () => {
     const [notificationModalData, setNotificationModalData] = React.useState(null);
     const [isLoading, setIsLoading] = React.useState(false);
 
@@ -38,4 +38,4 @@ const useFetchModalData = () => {
     return { notificationModalData, isLoading };
 };
 
-module.exports = useFetchModalData;
+module.exports = useFetchNotificationData;