fix double modal hide callbacks when unmounted

Author: bernhardoj

src/components/Modal/BaseModal.tsx

@@ -102,8 +102,6 @@ function BaseModal(
     const insets = useSafeAreaInsets();
 
     const isVisibleRef = useRef(isVisible);
-    const hideModalCallbackRef = useRef<(callHideCallback: boolean) => void>(undefined);
-
     const wasVisible = usePrevious(isVisible);
 
     const uniqueModalId = useMemo(() => modalId ?? ComposerFocusManager.getId(), [modalId]);
@@ -153,22 +151,6 @@ function BaseModal(
         };
     }, [isVisible, wasVisible, onClose, type]);
 
-    useEffect(() => {
-        hideModalCallbackRef.current = hideModal;
-    }, [hideModal]);
-
-    useEffect(
-        () => () => {
-            // Only trigger onClose and setModalVisibility if the modal is unmounting while visible.
-            if (!isVisibleRef.current) {
-                return;
-            }
-            hideModalCallbackRef.current?.(true);
-        },
-        // eslint-disable-next-line react-compiler/react-compiler, react-hooks/exhaustive-deps
-        [],
-    );
-
     const handleShowModal = useCallback(() => {
         if (shouldSetModalVisibility) {
             setModalVisibility(true, type);