From fef1d5cadb68c16fb6c8bda2c586dec95d3e7a25 Mon Sep 17 00:00:00 2001
From: Jonas Honecker <honeckerjonas@gmail.com>
Date: Fri, 5 Jul 2024 13:25:22 +0200
Subject: [PATCH] Refactoring: use one status instead of two for visuals about
 request status

---
 frontend/src/components/forms/SearchForm.tsx   | 10 +++++-----
 frontend/src/components/navbars/MainNavBar.tsx |  8 ++++----
 frontend/src/components/pages/MainPage.tsx     |  5 ++---
 3 files changed, 11 insertions(+), 12 deletions(-)

diff --git a/frontend/src/components/forms/SearchForm.tsx b/frontend/src/components/forms/SearchForm.tsx
index 878a2d1..69d6ac7 100644
--- a/frontend/src/components/forms/SearchForm.tsx
+++ b/frontend/src/components/forms/SearchForm.tsx
@@ -8,24 +8,24 @@ import { SnackbarConfig } from "../../types/Config.ts";
 type SearchFormProps = {
   setSearchResults: Dispatch<SetStateAction<Ticket[] | undefined>>;
   setSnackbarConfig: Dispatch<SetStateAction<SnackbarConfig>>;
-  setLoadingTickets: Dispatch<SetStateAction<boolean>>;
+  setPendingRequest: Dispatch<SetStateAction<boolean>>;
 };
 
 export default function SearchForm({
   setSearchResults,
   setSnackbarConfig,
-  setLoadingTickets,
+  setPendingRequest,
 }: Readonly<SearchFormProps>) {
   function searchTickets(event: FormEvent) {
     event.preventDefault();
-    setLoadingTickets(true);
+    setPendingRequest(true);
     getAllTickets(searchText)
       .then((response) => {
-        setLoadingTickets(false);
+        setPendingRequest(false);
         setSearchResults(response.data);
       })
       .catch((error) => {
-        setLoadingTickets(false);
+        setPendingRequest(false);
         setSnackbarConfig({
           open: true,
           severity: "error",
diff --git a/frontend/src/components/navbars/MainNavBar.tsx b/frontend/src/components/navbars/MainNavBar.tsx
index 3a7a09f..6295bbb 100644
--- a/frontend/src/components/navbars/MainNavBar.tsx
+++ b/frontend/src/components/navbars/MainNavBar.tsx
@@ -9,13 +9,13 @@ import { SidepanelConfig } from "../../types/Config.ts";
 type NavBarProps = {
   user: User | null | undefined;
   setSidepanelConfig: Dispatch<SetStateAction<SidepanelConfig>>;
-  loadingTickets: boolean;
+  pendingRequest: boolean;
 };
 
 export default function MainNavBar({
   user,
   setSidepanelConfig,
-  loadingTickets,
+  pendingRequest,
 }: Readonly<NavBarProps>) {
   return (
     <AppBar elevation={0} position={"sticky"}>
@@ -26,8 +26,8 @@ export default function MainNavBar({
         </Box>
         {user && <UserMenuButton user={user} />}
       </Toolbar>
-      {loadingTickets && (
-        <Fade in={loadingTickets}>
+      {pendingRequest && (
+        <Fade in={pendingRequest}>
           <Box sx={{ position: "absolute", bottom: "-4px", width: "100%" }}>
             <LinearProgress />
           </Box>
diff --git a/frontend/src/components/pages/MainPage.tsx b/frontend/src/components/pages/MainPage.tsx
index fd6279c..299bbf7 100644
--- a/frontend/src/components/pages/MainPage.tsx
+++ b/frontend/src/components/pages/MainPage.tsx
@@ -32,7 +32,6 @@ export default function MainPage({
     message: "Initial value",
   });
   const [confirmDeletion, setConfirmDeletion] = useState(false);
-  const [loadingTickets, setLoadingTickets] = useState(false);
   const [pendingRequest, setPendingRequest] = useState<boolean>(false);
 
   return (
@@ -40,14 +39,14 @@ export default function MainPage({
       <MainNavBar
         user={user}
         setSidepanelConfig={setSidepanelConfig}
-        loadingTickets={loadingTickets}
+        pendingRequest={pendingRequest}
       />
       <Container fixed sx={{ p: 3 }}>
         <Box sx={{ display: "flex", justifyContent: "center", mb: 3 }}>
           <SearchForm
             setSearchResults={setSearchResults}
             setSnackbarConfig={setSnackbarConfig}
-            setLoadingTickets={setLoadingTickets}
+            setPendingRequest={setPendingRequest}
           />
         </Box>
         <TicketCardsGrid