From 9ef1b2d8235f7e1194d2c8f5b2df93ab4bcc81c2 Mon Sep 17 00:00:00 2001
From: "fredy.siswanto" <fredy.siswanto@komunal.co.id>
Date: Sun, 23 Feb 2025 15:11:35 +0700
Subject: [PATCH] feat: rename success state variables for consistency and
 clarity in News context

---
 app/components/ui/form-subscription.tsx |  2 +-
 app/contexts/news.tsx                   | 10 +++++-----
 app/layouts/news/default.tsx            |  8 ++++----
 3 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/app/components/ui/form-subscription.tsx b/app/components/ui/form-subscription.tsx
index b65addf..99ce7e8 100644
--- a/app/components/ui/form-subscription.tsx
+++ b/app/components/ui/form-subscription.tsx
@@ -1,4 +1,4 @@
-import { Button } from '@headlessui/react'
+import { Button } from '~/components/ui/button'
 
 export default function FormSubscription() {
   return (
diff --git a/app/contexts/news.tsx b/app/contexts/news.tsx
index bedf5a7..d830ad5 100644
--- a/app/contexts/news.tsx
+++ b/app/contexts/news.tsx
@@ -14,8 +14,8 @@ type NewsContextProperties = {
   setIsRegisterOpen: Dispatch<SetStateAction<boolean>>
   isForgetOpen: boolean
   setForgetOpen: Dispatch<SetStateAction<boolean>>
-  isSuccessOpen: boolean
-  setIsSuccessOpen: Dispatch<SetStateAction<boolean>>
+  isSuccessModalOpen: boolean
+  setIsSuccessModalOpen: Dispatch<SetStateAction<boolean>>
 }
 
 const NewsContext = createContext<NewsContextProperties | undefined>(undefined)
@@ -24,7 +24,7 @@ export const NewsProvider = ({ children }: PropsWithChildren) => {
   const [isLoginOpen, setIsLoginOpen] = useState(false)
   const [isRegisterOpen, setIsRegisterOpen] = useState(false)
   const [isForgetOpen, setForgetOpen] = useState(false)
-  const [isSuccessOpen, setIsSuccessOpen] = useState(true)
+  const [isSuccessModalOpen, setIsSuccessModalOpen] = useState(false)
 
   return (
     <NewsContext.Provider
@@ -35,8 +35,8 @@ export const NewsProvider = ({ children }: PropsWithChildren) => {
         setIsRegisterOpen,
         isForgetOpen,
         setForgetOpen,
-        isSuccessOpen,
-        setIsSuccessOpen,
+        isSuccessModalOpen,
+        setIsSuccessModalOpen,
       }}
     >
       {children}
diff --git a/app/layouts/news/default.tsx b/app/layouts/news/default.tsx
index 12a1327..58fe200 100644
--- a/app/layouts/news/default.tsx
+++ b/app/layouts/news/default.tsx
@@ -22,8 +22,8 @@ export const NewsDefaultLayout = (properties: PropsWithChildren) => {
     setIsRegisterOpen,
     isForgetOpen,
     setForgetOpen,
-    isSuccessOpen,
-    setIsSuccessOpen,
+    isSuccessModalOpen,
+    setIsSuccessModalOpen,
   } = useNewsContext()
   return (
     <main className="relative min-h-dvh bg-[#ECECEC]">
@@ -70,9 +70,9 @@ export const NewsDefaultLayout = (properties: PropsWithChildren) => {
       </PopupModal>
 
       <SuccessModal
-        isOpen={isSuccessOpen}
+        isOpen={isSuccessModalOpen}
         onClose={() => {
-          setIsSuccessOpen(false)
+          setIsSuccessModalOpen(false)
         }}
         type="warning"
       />