fix: rename shouldCountStreak to shouldIcrementStreak and update date comparison logic

Author: medyo

src/components/Layout/AppLayout.tsx

@@ -11,17 +11,17 @@ import { AuthProvider } from 'src/providers/AuthProvider'
 import { Header } from './Header'
 
 export const AppLayout = () => {
-  const { isAuthModalOpen, setStreak, shouldCountStreak } = useAuth()
+  const { isAuthModalOpen, setStreak, shouldIcrementStreak } = useAuth()
   const postStreakMutation = usePostStreak()
 
   useEffect(() => {
-    if (shouldCountStreak()) {
+    if (shouldIcrementStreak()) {
       postStreakMutation.mutateAsync(undefined).then((data) => {
         setStreak(data.streak)
         identifyUserStreak(data.streak)
       })
     }
-  }, [shouldCountStreak])
+  }, [])
 
   return (
     <AuthProvider>

src/features/auth/hooks/useAuth.ts

@@ -10,14 +10,14 @@ export const useAuth = () => {
 
   const isConnected = authStore.user != null
 
-  const shouldCountStreak = useCallback(() => {
+  const shouldIcrementStreak = useCallback(() => {
     if (!isConnected) return false
 
     const last = authStore.lastStreakUpdate
     if (!last) return true
 
-    const today = new Date().toDateString()
-    const lastDay = new Date(last).toDateString()
+    const today = new Date().toISOString()
+    const lastDay = new Date(last).toISOString()
 
     return today !== lastDay
   }, [isConnected, authStore.lastStreakUpdate])
@@ -33,7 +33,7 @@ export const useAuth = () => {
     ...authModalStore,
     ...authStore,
     isConnected,
-    shouldCountStreak,
+    shouldIcrementStreak,
     logout,
   }
 }