Merge pull request Expensify#53111 from nkdengineer/fix/52415

fix: user lands on previous attachment
bernhardoj · Dec 11, 2024 · f8f12c2 · f8f12c2
2 parents f9ec4fa + cf78559
commit f8f12c2
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/src/components/Attachments/AttachmentCarousel/Pager/index.tsx b/src/components/Attachments/AttachmentCarousel/Pager/index.tsx
@@ -85,10 +85,7 @@ function AttachmentCarouselPager(
         [activePageIndex, items],
     );
 
-    const extractItemKey = useCallback(
-        (item: Attachment) => (typeof item.source === 'string' || typeof item.source === 'number' ? `source-${item.source}|${item.attachmentLink}` : `reportActionID-${item.reportActionID}`),
-        [],
-    );
+    const extractItemKey = useCallback((item: Attachment, index: number) => `reportActionID-${item.reportActionID}-${index}`, []);
 
     const contextValue = useMemo(
         () => ({
@@ -124,7 +121,7 @@ function AttachmentCarouselPager(
 
     const carouselItems = items.map((item, index) => (
         <View
-            key={extractItemKey(item)}
+            key={extractItemKey(item, index)}
             style={styles.flex1}
         >
             <CarouselItem