Fix event deletion bug

src/events/new/NewEventDialog.tsx

@@ -10,6 +10,7 @@ import { collections } from '../../services/firebase'
 import LoadingButton from '@mui/lab/LoadingButton'
 import { NewEvent } from '../../types'
 import { serverTimestamp } from 'firebase/firestore'
+import { useNotification } from '../../hooks/notificationHook'
 
 export type NewEventDialogProps = {
     isOpen: boolean
@@ -25,6 +26,7 @@ const schema = yup
 export const NewEventDialog = ({ isOpen, onClose }: NewEventDialogProps) => {
     const userId = useSelector(selectUserIdOpenPlanner)
 
+    const { createNotification } = useNotification()
     const formContext = useForm()
     const { formState } = formContext
 
@@ -66,8 +68,8 @@ export const NewEventDialog = ({ isOpen, onClose }: NewEventDialogProps) => {
                             onClose(eventId)
                         })
                         .catch((error: Error) => {
+                            createNotification('Error while creating event: ' + error.message, { type: 'error' })
                             console.error('error creating event', error)
-                            alert('Error creating event: ' + error.message)
                         })
                 }}>
                 <DialogContent>

src/events/page/settings/EventSettings.tsx

@@ -190,9 +190,9 @@ export const EventSettings = ({ event }: EventSettingsProps) => {
                     handleAccept={async () => {
                         await deleteSessionsAndSpeakers(event, false)
                         await documentDeletion.mutate()
-                        setLocation('../../')
                         setDeleteOpen(false)
                         createNotification('Event deleted', { type: 'success' })
+                        setLocation('../../')
                     }}>
                     <DialogContentText id="alert-dialog-description">
                         {' '}

src/services/hooks/firestoreQueryHook.ts

@@ -60,9 +60,7 @@ export const useFirestoreCollection = <T>(query: Query<T>, subscribe: boolean =
     }, [])
 
     useEffect(() => {
-        const unsubFunc = load()
-
-        return unsubFunc
+        return load()
     }, [load])
 
     return {
@@ -82,9 +80,15 @@ export const useFirestoreDocument = <T>(ref: DocumentReference<T>, subscribe: bo
     const isError = error !== ''
 
     const docTransformer = (querySnapshot: DocumentSnapshot<T, DocumentData>) => {
+        const d = querySnapshot.data()
+
+        if (d === undefined) {
+            return
+        }
+
         setData({
             id: querySnapshot.id,
-            ...(querySnapshot.data() as T),
+            ...(d as T),
         })
     }
 
@@ -114,9 +118,7 @@ export const useFirestoreDocument = <T>(ref: DocumentReference<T>, subscribe: bo
     }, [])
 
     useEffect(() => {
-        const unsubFunc = load()
-
-        return unsubFunc
+        return load()
     }, [])
 
     return {

src/services/hooks/useEvent.ts

@@ -1,8 +1,10 @@
 import { collections } from '../firebase'
 import { doc, DocumentData } from '@firebase/firestore'
 import { useFirestoreDocument, UseQueryResult } from './firestoreQueryHook'
+import { DocumentReference } from 'firebase/firestore'
+import { Event } from '../../types'
 
 export const useEvent = (eventId?: string): UseQueryResult<DocumentData> => {
-    const ref = doc(collections.events, eventId)
-    return useFirestoreDocument(ref, true)
+    const ref: DocumentReference<Event> = doc(collections.events, eventId) as DocumentReference<Event>
+    return useFirestoreDocument<Event>(ref, true)
 }