fix: CI checks

Expensify · Feb 22, 2024 · 44efead · 44efead
1 parent 20f51ca
commit 44efead
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 5 deletions.
diff --git a/src/components/Attachments/AttachmentCarousel/Pager/index.tsx b/src/components/Attachments/AttachmentCarousel/Pager/index.tsx
@@ -42,12 +42,15 @@ type AttachmentCarouselPagerProps = {
      * @param showArrows If set, it will show/hide the arrows. If not set, it will toggle the arrows.
      */
     onRequestToggleArrows: (showArrows?: boolean) => void;
-    
+
     /** A callback that is called when swipe-down-to-close gesture happens */
     onClose: () => void;
 };
 
-function AttachmentCarouselPager({items, activeSource, initialPage, onPageSelected, onRequestToggleArrows, onClose}: AttachmentCarouselPagerProps, ref: ForwardedRef<AttachmentCarouselPagerHandle>) {
+function AttachmentCarouselPager(
+    {items, activeSource, initialPage, onPageSelected, onRequestToggleArrows, onClose}: AttachmentCarouselPagerProps,
+    ref: ForwardedRef<AttachmentCarouselPagerHandle>,
+) {
     const styles = useThemeStyles();
     const pagerRef = useRef<PagerView>(null);
 

diff --git a/src/components/MultiGestureCanvas/index.tsx b/src/components/MultiGestureCanvas/index.tsx
@@ -10,7 +10,7 @@ import useStyleUtils from '@hooks/useStyleUtils';
 import useThemeStyles from '@hooks/useThemeStyles';
 import type ChildrenProps from '@src/types/utils/ChildrenProps';
 import {DEFAULT_ZOOM_RANGE, SPRING_CONFIG, ZOOM_RANGE_BOUNCE_FACTORS} from './constants';
-import type {CanvasSize, ContentSize, OnScaleChangedCallback, OnTapCallback, ZoomRange, OnSwipeDownCallback} from './types';
+import type {CanvasSize, ContentSize, OnScaleChangedCallback, OnSwipeDownCallback, OnTapCallback, ZoomRange} from './types';
 import usePanGesture from './usePanGesture';
 import usePinchGesture from './usePinchGesture';
 import useTapGestures from './useTapGestures';

diff --git a/src/components/MultiGestureCanvas/types.ts b/src/components/MultiGestureCanvas/types.ts
@@ -49,7 +49,7 @@ type MultiGestureCanvasVariables = {
     reset: (animated: boolean, callback: () => void) => void;
     onTap: OnTapCallback | undefined;
     onScaleChanged: OnScaleChangedCallback | undefined;
-    onSwipeDown: OnSwipeDownCallback;
+    onSwipeDown: OnSwipeDownCallback | undefined;
 };
 
 export type {CanvasSize, ContentSize, ZoomRange, OnScaleChangedCallback, OnTapCallback, MultiGestureCanvasVariables, OnSwipeDownCallback};
diff --git a/src/components/MultiGestureCanvas/usePanGesture.ts b/src/components/MultiGestureCanvas/usePanGesture.ts
@@ -143,7 +143,9 @@ const usePanGesture = ({
                     isSwipingDownToClose.value = false;
                 });
 
-                runOnJS(onSwipeDown)();
+                if (onSwipeDown) {
+                    runOnJS(onSwipeDown)();
+                }
             } else {
                 // Animated back to the boundary
                 offsetY.value = withSpring(clampedOffset.y, SPRING_CONFIG, () => {