Fix bug showing cloud message always on settings > datasets

We were not checking if that message modal was open. This was part of a
refactor where this modal was in an independent route. Now state lives
in their parent component

Author: andresgutgon

apps/web/src/app/(private)/datasets/_components/RootHeader/GenerateDatasetModal/AutogenerateDatasetNotAvailableModal/index.tsx

@@ -1,18 +1,21 @@
-'use client'
-
 import { ConfirmModal } from '@latitude-data/web-ui'
-import { useNavigate } from '$/hooks/useNavigate'
-import { ROUTES } from '$/services/routes'
 import { CLOUD_MESSAGES } from '@latitude-data/core/browser'
 
-export function AutogenerateDatasetNotAvailableModal() {
-  const navigate = useNavigate()
+export function AutogenerateDatasetNotAvailableModal({
+  open,
+  onOpenChange,
+}: {
+  open: boolean
+
+  onOpenChange: (open: boolean) => void
+}) {
   return (
     <ConfirmModal
-      open
-      dismissible={false}
+      dismissible
+      open={open}
+      onOpenChange={onOpenChange}
       title='Auto Generate dataset'
-      onConfirm={() => navigate.push(ROUTES.datasets.root())}
+      onConfirm={() => onOpenChange(false)}
       confirm={{
         label: 'Back to datasets',
         description: CLOUD_MESSAGES.autogenerateDatasets,

apps/web/src/app/(private)/datasets/_components/RootHeader/GenerateDatasetModal/index.tsx

@@ -23,7 +23,13 @@ export function GenerateDatasetCloudModal({
   generateInput: GenerateInputProps
   canNotModify: boolean
 }) {
-  if (!isCloud) return <AutogenerateDatasetNotAvailableModal />
+  if (!isCloud)
+    return (
+      <AutogenerateDatasetNotAvailableModal
+        open={open}
+        onOpenChange={onOpenChange}
+      />
+    )
 
   if (canNotModify) {
     return (