fix: getDbInstance instead of closeDbInstance

gregnr · gregnr · commit c33c59c27c57 · 2024-08-05T17:11:11.000-05:00
diff --git a/apps/postgres-new/app/db/[id]/page.tsx b/apps/postgres-new/app/db/[id]/page.tsx
@@ -17,7 +17,7 @@ export default function Page({ params }: { params: { id: string } }) {
       }
 
       try {
-        await dbManager.closeDbInstance(databaseId)
+        await dbManager.getDbInstance(databaseId)
       } catch (err) {
         router.push('/')
       }
diff --git a/apps/postgres-new/components/app-provider.tsx b/apps/postgres-new/components/app-provider.tsx
@@ -20,6 +20,9 @@ import { createClient } from '~/utils/supabase/client'
 
 export type AppProps = PropsWithChildren
 
+// Create a singleton DbManager that isn't exposed to double mounting
+const dbManager = typeof window !== 'undefined' ? new DbManager() : undefined
+
 export default function AppProvider({ children }: AppProps) {
   const [isLoadingUser, setIsLoadingUser] = useState(true)
   const [user, setUser] = useState<User>()
@@ -77,10 +80,6 @@ export default function AppProvider({ children }: AppProps) {
     setUser(undefined)
   }, [supabase])
 
-  const dbManager = useMemo(() => {
-    return typeof window !== 'undefined' ? new DbManager() : undefined
-  }, [])
-
   const isPreview = process.env.NEXT_PUBLIC_IS_PREVIEW === 'true'
   const pgliteVersion = process.env.NEXT_PUBLIC_PGLITE_VERSION
   const { value: pgVersion } = useAsyncMemo(async () => {

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ export default function Page({ params }: { params: { id: string } }) {`
`17`	`17`	`}`
`18`	`18`
`19`	`19`	`try {`
`20`		`- await dbManager.closeDbInstance(databaseId)`
	`20`	`+ await dbManager.getDbInstance(databaseId)`
`21`	`21`	`} catch (err) {`
`22`	`22`	`router.push('/')`
`23`	`23`	`}`