Skip to content

Commit e095e07

Browse files
committed
dynamic route for isQuickSettingsFlow
1 parent 013feee commit e095e07

8 files changed

Lines changed: 3 additions & 25 deletions

File tree

src/ROUTES.ts

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,7 @@ const DYNAMIC_ROUTES = {
295295
},
296296
SPEND_CATEGORY_SELECTOR: {
297297
path: 'spend-category-selector/:groupID',
298-
entryScreens: [SCREENS.WORKSPACE.CATEGORIES_SETTINGS],
298+
entryScreens: [SCREENS.WORKSPACE.CATEGORIES_SETTINGS, SCREENS.SETTINGS_CATEGORIES.SETTINGS_CATEGORIES_SETTINGS],
299299
getRoute: (groupID: string) => `spend-category-selector/${groupID}` as const,
300300
},
301301
DEFAULT_CATEGORY_SELECTOR: {
@@ -1527,12 +1527,6 @@ const ROUTES = {
15271527
// eslint-disable-next-line no-restricted-syntax -- Legacy route generation
15281528
getRoute: (policyID: string, backTo = '') => getUrlWithBackToParam(`settings/${policyID}/categories/new`, backTo),
15291529
},
1530-
SETTINGS_SPEND_CATEGORY_SELECTOR: {
1531-
route: 'settings/:policyID/categories/spend-category-selector/:groupID',
1532-
getRoute: (policyID: string, groupID: string, backTo = '') =>
1533-
// eslint-disable-next-line no-restricted-syntax -- Legacy route generation
1534-
getUrlWithBackToParam(`settings/${policyID}/categories/spend-category-selector/${groupID}` as const, backTo),
1535-
},
15361530
SETTINGS_CATEGORIES_IMPORT: {
15371531
route: 'settings/:policyID/categories/import',
15381532

src/SCREENS.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -429,7 +429,6 @@ const SCREENS = {
429429
SETTINGS_CATEGORIES_SETTINGS: 'Settings_Categories_Settings',
430430
SETTINGS_CATEGORY_CREATE: 'Settings_Category_Create',
431431
DYNAMIC_SETTINGS_CATEGORY_EDIT: 'Dynamic_Settings_Category_Edit',
432-
SETTINGS_SPEND_CATEGORY_SELECTOR: 'Settings_Spend_Category_Selector',
433432
SETTINGS_CATEGORIES_ROOT: 'Settings_Categories',
434433
SETTINGS_CATEGORIES_IMPORT: 'Settings_Categories_Import',
435434
SETTINGS_CATEGORIES_IMPORTED: 'Settings_Categories_Imported',

src/libs/Navigation/AppNavigator/ModalStackNavigators/index.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -315,7 +315,6 @@ const CategoriesModalStackNavigator = createModalStackNavigator({
315315
[SCREENS.SETTINGS_CATEGORIES.SETTINGS_CATEGORIES_SETTINGS]: () => require<ReactComponentModule>('../../../../pages/workspace/categories/WorkspaceCategoriesSettingsPage').default,
316316
[SCREENS.SETTINGS_CATEGORIES.SETTINGS_CATEGORY_CREATE]: () => require<ReactComponentModule>('../../../../pages/workspace/categories/CreateCategoryPage').default,
317317
[SCREENS.SETTINGS_CATEGORIES.DYNAMIC_SETTINGS_CATEGORY_EDIT]: () => require<ReactComponentModule>('../../../../pages/workspace/categories/EditCategoryPage').default,
318-
[SCREENS.SETTINGS_CATEGORIES.SETTINGS_SPEND_CATEGORY_SELECTOR]: () => require<ReactComponentModule>('../../../../pages/workspace/categories/DynamicSpendCategorySelectorPage').default,
319318
[SCREENS.SETTINGS_CATEGORIES.SETTINGS_CATEGORY_SETTINGS]: () => require<ReactComponentModule>('../../../../pages/workspace/categories/CategorySettingsPage').default,
320319
[SCREENS.SETTINGS_CATEGORIES.SETTINGS_CATEGORIES_IMPORT]: () => require<ReactComponentModule>('../../../../pages/workspace/categories/ImportCategoriesPage').default,
321320
[SCREENS.SETTINGS_CATEGORIES.SETTINGS_CATEGORIES_IMPORTED]: () => require<ReactComponentModule>('../../../../pages/workspace/categories/ImportedCategoriesPage').default,

src/libs/Navigation/linkingConfig/RELATIONS/WORKSPACE_TO_RHP.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -211,7 +211,6 @@ const WORKSPACE_TO_RHP: Partial<Record<keyof WorkspaceSplitNavigatorParamList, s
211211
SCREENS.WORKSPACE.CATEGORY_REQUIRE_RECEIPTS_OVER,
212212
SCREENS.WORKSPACE.CATEGORY_REQUIRE_ITEMIZED_RECEIPTS_OVER,
213213
SCREENS.WORKSPACE.CATEGORY_REQUIRED_FIELDS,
214-
SCREENS.WORKSPACE.DYNAMIC_SPEND_CATEGORY_SELECTOR,
215214
],
216215
[SCREENS.WORKSPACE.DISTANCE_RATES]: [
217216
SCREENS.WORKSPACE.DISTANCE_RATES_SETTINGS,

src/libs/Navigation/linkingConfig/config.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1480,9 +1480,6 @@ const config: LinkingOptions<RootNavigatorParamList>['config'] = {
14801480
path: ROUTES.SETTINGS_CATEGORY_CREATE.route,
14811481
},
14821482
[SCREENS.SETTINGS_CATEGORIES.DYNAMIC_SETTINGS_CATEGORY_EDIT]: DYNAMIC_ROUTES.SETTINGS_CATEGORY_EDIT.path,
1483-
[SCREENS.SETTINGS_CATEGORIES.SETTINGS_SPEND_CATEGORY_SELECTOR]: {
1484-
path: ROUTES.SETTINGS_SPEND_CATEGORY_SELECTOR.route,
1485-
},
14861483
[SCREENS.SETTINGS_CATEGORIES.SETTINGS_CATEGORIES_IMPORT]: {
14871484
path: ROUTES.SETTINGS_CATEGORIES_IMPORT.route,
14881485
},

src/libs/Navigation/types.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -442,10 +442,6 @@ type SettingsNavigatorParamList = {
442442
policyID: string;
443443
groupID: string;
444444
};
445-
[SCREENS.SETTINGS_CATEGORIES.SETTINGS_SPEND_CATEGORY_SELECTOR]: {
446-
policyID: string;
447-
groupID: string;
448-
};
449445
[SCREENS.WORKSPACE.DYNAMIC_DEFAULT_CATEGORY_SELECTOR]: {
450446
policyID: string;
451447
customUnitID: string;

src/pages/workspace/categories/DynamicSpendCategorySelectorPage.tsx

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,7 @@ import ONYXKEYS from '@src/ONYXKEYS';
1414
import {DYNAMIC_ROUTES} from '@src/ROUTES';
1515
import type SCREENS from '@src/SCREENS';
1616

17-
type DynamicSpendCategorySelectorPageProps =
18-
| PlatformStackScreenProps<SettingsNavigatorParamList, typeof SCREENS.WORKSPACE.DYNAMIC_SPEND_CATEGORY_SELECTOR>
19-
| PlatformStackScreenProps<SettingsNavigatorParamList, typeof SCREENS.SETTINGS_CATEGORIES.SETTINGS_SPEND_CATEGORY_SELECTOR>;
17+
type DynamicSpendCategorySelectorPageProps = PlatformStackScreenProps<SettingsNavigatorParamList, typeof SCREENS.WORKSPACE.DYNAMIC_SPEND_CATEGORY_SELECTOR>;
2018

2119
function DynamicSpendCategorySelectorPage({route}: DynamicSpendCategorySelectorPageProps) {
2220
const {policyID, groupID} = route.params;

src/pages/workspace/categories/WorkspaceCategoriesSettingsPage.tsx

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -73,11 +73,7 @@ function WorkspaceCategoriesSettingsPage({policy, route}: WorkspaceCategoriesSet
7373
return;
7474
}
7575

76-
Navigation.navigate(
77-
isQuickSettingsFlow
78-
? ROUTES.SETTINGS_SPEND_CATEGORY_SELECTOR.getRoute(policyID, item.groupID, backTo)
79-
: createDynamicRoute(DYNAMIC_ROUTES.SPEND_CATEGORY_SELECTOR.getRoute(item.groupID)),
80-
);
76+
Navigation.navigate(createDynamicRoute(DYNAMIC_ROUTES.SPEND_CATEGORY_SELECTOR.getRoute(item.groupID)));
8177
};
8278

8379
const selectionListHeaderContent = (

0 commit comments

Comments
 (0)