Merge pull request #58278 from callstack-internal/fix/57744-deep-link-check-report-id

youssef-lr · web-flow · commit a3c591ba9b4f · 2025-03-13T18:06:01.000Z
Fix reportID condition when SSO redirect
diff --git a/src/libs/Navigation/helpers/linkTo/index.ts b/src/libs/Navigation/helpers/linkTo/index.ts
@@ -62,7 +62,7 @@ function isNavigatingToReportWithSameReportID(currentRoute: NavigationPartialRou
     const currentParams = currentRoute.params as ReportsSplitNavigatorParamList[typeof SCREENS.REPORT];
     const newParams = newRoute?.params as ReportsSplitNavigatorParamList[typeof SCREENS.REPORT];
 
-    return currentParams.reportID === newParams.reportID;
+    return currentParams?.reportID === newParams?.reportID;
 }
 
 export default function linkTo(navigation: NavigationContainerRef<RootNavigatorParamList> | null, path: Route, options?: LinkToOptions) {

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ function isNavigatingToReportWithSameReportID(currentRoute: NavigationPartialRou`
`62`	`62`	`const currentParams = currentRoute.params as ReportsSplitNavigatorParamList[typeof SCREENS.REPORT];`
`63`	`63`	`const newParams = newRoute?.params as ReportsSplitNavigatorParamList[typeof SCREENS.REPORT];`
`64`	`64`
`65`		`- return currentParams.reportID === newParams.reportID;`
	`65`	`+ return currentParams?.reportID === newParams?.reportID;`
`66`	`66`	`}`
`67`	`67`
`68`	`68`	`export default function linkTo(navigation: NavigationContainerRef<RootNavigatorParamList> \| null, path: Route, options?: LinkToOptions) {`