Skip to content

Commit

Permalink
fix: 30988 New dot opens when navigate to the staging magic link, not…
Browse files Browse the repository at this point in the history
… abracadabra page
  • Loading branch information
tienifr committed Nov 7, 2023
1 parent ece8312 commit a24d9f0
Showing 1 changed file with 12 additions and 16 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -26,22 +26,21 @@ const defaultProps = {
screenOptions: undefined,
};

function splitRoutes(routes) {
const reportRoutes = [];
const rhpRoutes = [];
const otherRoutes = [];

function reduceReportRoutes(routes) {
const result = [];
let count = 0;
routes.forEach((route) => {
if (route.name === NAVIGATORS.CENTRAL_PANE_NAVIGATOR) {
reportRoutes.push(route);
} else if (route.name === NAVIGATORS.RIGHT_MODAL_NAVIGATOR) {
rhpRoutes.push(route);
if (count < 3) {
result.push(route);
count++;
}
} else {
otherRoutes.push(route);
result.push(route);
}
});

return {reportRoutes, rhpRoutes, otherRoutes};
return result;
}

function ResponsiveStackNavigator(props) {
Expand All @@ -60,15 +59,12 @@ function ResponsiveStackNavigator(props) {
});

const stateToRender = useMemo(() => {
const {reportRoutes, rhpRoutes, otherRoutes} = splitRoutes(state.routes);

// Remove all report routes except the last 3. This will improve performance.
const limitedReportRoutes = reportRoutes.slice(-3);
const result = reduceReportRoutes(state.routes);

return {
...state,
index: otherRoutes.length + limitedReportRoutes.length + rhpRoutes.length - 1,
routes: [...otherRoutes, ...limitedReportRoutes, ...rhpRoutes],
index: result.length - 1,
routes: [...result],
};
}, [state]);

Expand Down

0 comments on commit a24d9f0

Please sign in to comment.