From 74e7d456aa3e00e5bbad4fa0345b039a65eb3075 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Tue, 5 Mar 2024 12:29:20 +0100
Subject: [PATCH 01/32] feat(bulk actions ui): add menu dropdown button

---
 src/CONST.ts                                  |  5 ++
 src/languages/en.ts                           |  5 +-
 src/languages/es.ts                           |  5 +-
 src/pages/workspace/WorkspaceMembersPage.tsx  |  2 +-
 .../categories/WorkspaceCategoriesPage.tsx    | 85 +++++++++++++++----
 5 files changed, 81 insertions(+), 21 deletions(-)

diff --git a/src/CONST.ts b/src/CONST.ts
index 9a9294621338..60d1b4ead7fb 100755
--- a/src/CONST.ts
+++ b/src/CONST.ts
@@ -1399,6 +1399,11 @@ const CONST = {
             MAKE_MEMBER: 'makeMember',
             MAKE_ADMIN: 'makeAdmin',
         },
+        CATEGORIES_BULK_ACTION_TYPES: {
+            DELETE: 'delete',
+            DISABLE: 'disable',
+            ENABLE: 'enable',
+        },
     },
 
     CUSTOM_UNITS: {
diff --git a/src/languages/en.ts b/src/languages/en.ts
index 0a52cca62ef5..6a5c67fb0902 100755
--- a/src/languages/en.ts
+++ b/src/languages/en.ts
@@ -1743,6 +1743,7 @@ export default {
             testTransactions: 'Test transactions',
             issueAndManageCards: 'Issue and manage cards',
             reconcileCards: 'Reconcile cards',
+            selected: ({selectedNumber}) => `${selectedNumber} selected`,
             settlementFrequency: 'Settlement frequency',
             deleteConfirmation: 'Are you sure you want to delete this workspace?',
             unavailable: 'Unavailable workspace',
@@ -1761,6 +1762,9 @@ export default {
             collect: 'Collect',
         },
         categories: {
+            deleteCategories: 'Delete categories',
+            disableCategories: 'Disable categories',
+            enableCategories: 'Enable categories',
             categoryName: 'Category name',
             requiresCategory: 'Members must categorize all spend',
             enableCategory: 'Enable category',
@@ -1808,7 +1812,6 @@ export default {
             makeMember: 'Make member',
             makeAdmin: 'Make admin',
             selectAll: 'Select all',
-            selected: ({selectedNumber}) => `${selectedNumber} selected`,
             error: {
                 genericAdd: 'There was a problem adding this workspace member.',
                 cannotRemove: 'You cannot remove yourself or the workspace owner.',
diff --git a/src/languages/es.ts b/src/languages/es.ts
index 013255c1e11e..7841846f465a 100644
--- a/src/languages/es.ts
+++ b/src/languages/es.ts
@@ -1767,6 +1767,7 @@ export default {
             testTransactions: 'Transacciones de prueba',
             issueAndManageCards: 'Emitir y gestionar tarjetas',
             reconcileCards: 'Reconciliar tarjetas',
+            selected: ({selectedNumber}) => `${selectedNumber} seleccionados`,
             settlementFrequency: 'Frecuencia de liquidación',
             deleteConfirmation: '¿Estás seguro de que quieres eliminar este espacio de trabajo?',
             unavailable: 'Espacio de trabajo no disponible',
@@ -1785,6 +1786,9 @@ export default {
             collect: 'Recolectar',
         },
         categories: {
+            deleteCategories: 'Eliminar categorías',
+            disableCategories: 'Desactivar categorías',
+            enableCategories: 'Activar categorías',
             categoryName: 'Nombre de la categoría',
             requiresCategory: 'Los miembros deben categorizar todos los gastos',
             enableCategory: 'Activar categoría',
@@ -1832,7 +1836,6 @@ export default {
             makeMember: 'Hacer miembro',
             makeAdmin: 'Hacer administrador',
             selectAll: 'Seleccionar todo',
-            selected: ({selectedNumber}) => `${selectedNumber} seleccionados`,
             error: {
                 genericAdd: 'Ha ocurrido un problema al añadir el miembro al espacio de trabajo.',
                 cannotRemove: 'No puedes eliminarte ni a ti mismo ni al dueño del espacio de trabajo.',
diff --git a/src/pages/workspace/WorkspaceMembersPage.tsx b/src/pages/workspace/WorkspaceMembersPage.tsx
index b43659a776d9..f532697ff7b1 100644
--- a/src/pages/workspace/WorkspaceMembersPage.tsx
+++ b/src/pages/workspace/WorkspaceMembersPage.tsx
@@ -434,7 +434,7 @@ function WorkspaceMembersPage({policyMembers, personalDetails, route, policy, se
                 <ButtonWithDropdownMenu<WorkspaceMemberBulkActionType>
                     shouldAlwaysShowDropdownMenu
                     pressOnEnter
-                    customText={translate('workspace.people.selected', {selectedNumber: selectedEmployees.length})}
+                    customText={translate('workspace.common.selected', {selectedNumber: selectedEmployees.length})}
                     buttonSize={CONST.DROPDOWN_BUTTON_SIZE.MEDIUM}
                     onPress={() => null}
                     options={getBulkActionsButtonOptions()}
diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index d15011489bac..3b3f2cc87f1c 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -1,9 +1,11 @@
 import type {StackScreenProps} from '@react-navigation/stack';
-import React, {useMemo, useState} from 'react';
+import React, {useCallback, useMemo, useRef, useState} from 'react';
 import {View} from 'react-native';
 import {withOnyx} from 'react-native-onyx';
 import type {OnyxEntry} from 'react-native-onyx';
 import Button from '@components/Button';
+import ButtonWithDropdownMenu from '@components/ButtonWithDropdownMenu';
+import type {DropdownOption} from '@components/ButtonWithDropdownMenu/types';
 import HeaderWithBackButton from '@components/HeaderWithBackButton';
 import Icon from '@components/Icon';
 import * as Expensicons from '@components/Icon/Expensicons';
@@ -21,10 +23,12 @@ import Navigation from '@libs/Navigation/Navigation';
 import type {CentralPaneNavigatorParamList} from '@navigation/types';
 import AdminPolicyAccessOrNotFoundWrapper from '@pages/workspace/AdminPolicyAccessOrNotFoundWrapper';
 import PaidPolicyAccessOrNotFoundWrapper from '@pages/workspace/PaidPolicyAccessOrNotFoundWrapper';
+import CONST from '@src/CONST';
 import ONYXKEYS from '@src/ONYXKEYS';
 import ROUTES from '@src/ROUTES';
 import type SCREENS from '@src/SCREENS';
 import type * as OnyxTypes from '@src/types/onyx';
+import type DeepValueOf from '@src/types/utils/DeepValueOf';
 
 type PolicyForList = {
     value: string;
@@ -47,6 +51,7 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
     const theme = useTheme();
     const {translate} = useLocalize();
     const [selectedCategories, setSelectedCategories] = useState<Record<string, boolean>>({});
+    const dropdownButtonRef = useRef(null);
 
     const categoryList = useMemo<PolicyForList[]>(
         () =>
@@ -89,25 +94,69 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
         </View>
     );
 
-    const navigateToCategoriesSettings = () => {
-        Navigation.navigate(ROUTES.WORKSPACE_CATEGORIES_SETTINGS.getRoute(route.params.policyID));
-    };
-
     const navigateToCategorySettings = (category: PolicyForList) => {
         Navigation.navigate(ROUTES.WORKSPACE_CATEGORY_SETTINGS.getRoute(route.params.policyID, category.text));
     };
 
-    const settingsButton = (
-        <View style={[styles.w100, styles.flexRow, isSmallScreenWidth && styles.mb3]}>
-            <Button
-                medium
-                onPress={navigateToCategoriesSettings}
-                icon={Expensicons.Gear}
-                text={translate('common.settings')}
-                style={[isSmallScreenWidth && styles.w50]}
-            />
-        </View>
-    );
+    const selectedCategoriesArray = Object.keys(selectedCategories).filter((key) => selectedCategories[key]);
+
+    const renderButtons = useCallback(() => {
+        if (selectedCategoriesArray.length > 0) {
+            const isAllEnabled = selectedCategoriesArray.every((category) => policyCategories?.[category].enabled);
+
+            const options: Array<DropdownOption<DeepValueOf<typeof CONST.POLICY.CATEGORIES_BULK_ACTION_TYPES>>> = [
+                {
+                    icon: Expensicons.Trashcan,
+                    text: translate('workspace.categories.deleteCategories'),
+                    value: CONST.POLICY.CATEGORIES_BULK_ACTION_TYPES.DELETE,
+                    onSelected: () => null,
+                },
+            ];
+
+            if (isAllEnabled) {
+                options.push({
+                    icon: Expensicons.Trashcan,
+                    text: translate('workspace.categories.disableCategories'),
+                    value: CONST.POLICY.CATEGORIES_BULK_ACTION_TYPES.DISABLE,
+                    onSelected: () => null,
+                });
+            } else {
+                options.push({
+                    icon: Expensicons.Trashcan,
+                    text: translate('workspace.categories.enableCategories'),
+                    value: CONST.POLICY.CATEGORIES_BULK_ACTION_TYPES.ENABLE,
+                    onSelected: () => null,
+                });
+            }
+
+            return (
+                <ButtonWithDropdownMenu<DeepValueOf<typeof CONST.POLICY.CATEGORIES_BULK_ACTION_TYPES>>
+                    buttonRef={dropdownButtonRef}
+                    onPress={() => null}
+                    shouldAlwaysShowDropdownMenu
+                    pressOnEnter
+                    buttonSize={CONST.DROPDOWN_BUTTON_SIZE.MEDIUM}
+                    customText={translate('workspace.common.selected', {selectedNumber: selectedCategoriesArray.length})}
+                    options={options}
+                />
+            );
+        }
+        const navigateToCategoriesSettings = () => {
+            Navigation.navigate(ROUTES.WORKSPACE_CATEGORIES_SETTINGS.getRoute(route.params.policyID));
+        };
+
+        return (
+            <View style={[styles.w100, styles.flexRow, isSmallScreenWidth && styles.mb3]}>
+                <Button
+                    medium
+                    onPress={navigateToCategoriesSettings}
+                    icon={Expensicons.Gear}
+                    text={translate('common.settings')}
+                    style={[isSmallScreenWidth && styles.w50]}
+                />
+            </View>
+        );
+    }, [isSmallScreenWidth, policyCategories, route.params.policyID, selectedCategoriesArray, styles.flexRow, styles.mb3, styles.w100, styles.w50, translate]);
 
     return (
         <AdminPolicyAccessOrNotFoundWrapper policyID={route.params.policyID}>
@@ -123,9 +172,9 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
                         title={translate('workspace.common.categories')}
                         shouldShowBackButton={isSmallScreenWidth}
                     >
-                        {!isSmallScreenWidth && settingsButton}
+                        {!isSmallScreenWidth && renderButtons()}
                     </HeaderWithBackButton>
-                    {isSmallScreenWidth && <View style={[styles.pl5, styles.pr5]}>{settingsButton}</View>}
+                    {isSmallScreenWidth && <View style={[styles.pl5, styles.pr5]}>{renderButtons()}</View>}
                     <View style={[styles.ph5, styles.pb5]}>
                         <Text style={[styles.textNormal, styles.colorMuted]}>{translate('workspace.categories.subtitle')}</Text>
                     </View>

From 3f6c8f56611131ae4f10359d7d48fa763b43cf8c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Tue, 5 Mar 2024 12:29:45 +0100
Subject: [PATCH 02/32] feat(onyx): implement bulk delete categories

---
 src/libs/API/parameters/index.ts |  1 +
 src/libs/API/types.ts            |  2 ++
 src/libs/actions/Policy.ts       | 40 ++++++++++++++++++++++++++++++++
 3 files changed, 43 insertions(+)

diff --git a/src/libs/API/parameters/index.ts b/src/libs/API/parameters/index.ts
index 679379c8089a..594e4175439d 100644
--- a/src/libs/API/parameters/index.ts
+++ b/src/libs/API/parameters/index.ts
@@ -149,6 +149,7 @@ export type {default as UpdateWorkspaceDescriptionParams} from './UpdateWorkspac
 export type {default as UpdateWorkspaceMembersRoleParams} from './UpdateWorkspaceMembersRoleParams';
 export type {default as SetWorkspaceCategoriesEnabledParams} from './SetWorkspaceCategoriesEnabledParams';
 export type {default as SetWorkspaceRequiresCategoryParams} from './SetWorkspaceRequiresCategoryParams';
+export type {default as DeleteWorkspaceCategoriesParams} from './DeleteWorkspaceCategoriesParams';
 export type {default as SetWorkspaceAutoReportingParams} from './SetWorkspaceAutoReportingParams';
 export type {default as SetWorkspaceAutoReportingFrequencyParams} from './SetWorkspaceAutoReportingFrequencyParams';
 export type {default as SetWorkspaceAutoReportingMonthlyOffsetParams} from './SetWorkspaceAutoReportingMonthlyOffsetParams';
diff --git a/src/libs/API/types.ts b/src/libs/API/types.ts
index cacbfebdc120..7a66c1d126aa 100644
--- a/src/libs/API/types.ts
+++ b/src/libs/API/types.ts
@@ -116,6 +116,7 @@ const WRITE_COMMANDS = {
     CREATE_WORKSPACE_FROM_IOU_PAYMENT: 'CreateWorkspaceFromIOUPayment',
     SET_WORKSPACE_CATEGORIES_ENABLED: 'SetWorkspaceCategoriesEnabled',
     SET_WORKSPACE_REQUIRES_CATEGORY: 'SetWorkspaceRequiresCategory',
+    DELETE_WORKSPACE_CATEGORIES: 'DeleteWorkspaceCategories',
     CREATE_TASK: 'CreateTask',
     CANCEL_TASK: 'CancelTask',
     EDIT_TASK_ASSIGNEE: 'EditTaskAssignee',
@@ -265,6 +266,7 @@ type WriteCommandParameters = {
     [WRITE_COMMANDS.CREATE_WORKSPACE_FROM_IOU_PAYMENT]: Parameters.CreateWorkspaceFromIOUPaymentParams;
     [WRITE_COMMANDS.SET_WORKSPACE_CATEGORIES_ENABLED]: Parameters.SetWorkspaceCategoriesEnabledParams;
     [WRITE_COMMANDS.SET_WORKSPACE_REQUIRES_CATEGORY]: Parameters.SetWorkspaceRequiresCategoryParams;
+    [WRITE_COMMANDS.DELETE_WORKSPACE_CATEGORIES]: Parameters.DeleteWorkspaceCategoriesParams;
     [WRITE_COMMANDS.CREATE_TASK]: Parameters.CreateTaskParams;
     [WRITE_COMMANDS.CANCEL_TASK]: Parameters.CancelTaskParams;
     [WRITE_COMMANDS.EDIT_TASK_ASSIGNEE]: Parameters.EditTaskAssigneeParams;
diff --git a/src/libs/actions/Policy.ts b/src/libs/actions/Policy.ts
index 313c4deb9934..7f0c50764ceb 100644
--- a/src/libs/actions/Policy.ts
+++ b/src/libs/actions/Policy.ts
@@ -2487,6 +2487,45 @@ function clearCategoryErrors(policyID: string, categoryName: string) {
     });
 }
 
+function deleteWorkspaceCategories(policyID: string, categoryNamesToDelete: string[]) {
+    const policyCategories = allPolicyCategories?.[`${ONYXKEYS.COLLECTION.POLICY_CATEGORIES}${policyID}`] ?? {};
+
+    const onyxData: OnyxData = {
+        optimisticData: [
+            {
+                onyxMethod: Onyx.METHOD.MERGE,
+                key: `${ONYXKEYS.COLLECTION.POLICY_CATEGORIES}${policyID}`,
+                value: {
+                    ...Object.keys(policyCategories).reduce<PolicyCategories>((acc, key) => {
+                        if (categoryNamesToDelete.includes(key)) {
+                            return acc;
+                        }
+
+                        acc[key] = policyCategories[key];
+                        return acc;
+                    }),
+                },
+            },
+        ],
+        failureData: [
+            {
+                onyxMethod: Onyx.METHOD.MERGE,
+                key: `${ONYXKEYS.COLLECTION.POLICY_CATEGORIES}${policyID}`,
+                value: {
+                    ...policyCategories,
+                },
+            },
+        ],
+    };
+
+    const parameters = {
+        policyID,
+        categories: categoryNamesToDelete,
+    };
+
+    API.write('DeleteWorkspaceCategories', parameters, onyxData);
+}
+
 export {
     removeMembers,
     updateWorkspaceMembersRole,
@@ -2536,4 +2575,5 @@ export {
     setWorkspaceCategoryEnabled,
     setWorkspaceRequiresCategory,
     clearCategoryErrors,
+    deleteWorkspaceCategories,
 };

From dfae8bc6942dee43ffe4a6e75d04f9d6458a0cd6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Tue, 5 Mar 2024 12:30:18 +0100
Subject: [PATCH 03/32] fix(onyx bulk delete): missing params type

---
 src/libs/API/parameters/DeleteWorkspaceCategoriesParams.ts | 6 ++++++
 1 file changed, 6 insertions(+)
 create mode 100644 src/libs/API/parameters/DeleteWorkspaceCategoriesParams.ts

diff --git a/src/libs/API/parameters/DeleteWorkspaceCategoriesParams.ts b/src/libs/API/parameters/DeleteWorkspaceCategoriesParams.ts
new file mode 100644
index 000000000000..f82841acdeba
--- /dev/null
+++ b/src/libs/API/parameters/DeleteWorkspaceCategoriesParams.ts
@@ -0,0 +1,6 @@
+type DeleteWorkspaceCategoriesParams = {
+    policyID: string;
+    categories: string[];
+};
+
+export default DeleteWorkspaceCategoriesParams;

From 10b282bdaf54ab43d6086561f390a1bf3326e96d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Tue, 5 Mar 2024 13:54:29 +0100
Subject: [PATCH 04/32] feat: connect bulk actions with onyx & api

---
 src/libs/actions/Policy.ts                    | 36 ++++++++++++++++--
 .../categories/WorkspaceCategoriesPage.tsx    | 37 +++++++++++++++++--
 2 files changed, 65 insertions(+), 8 deletions(-)

diff --git a/src/libs/actions/Policy.ts b/src/libs/actions/Policy.ts
index 7f0c50764ceb..48b87a234b6a 100644
--- a/src/libs/actions/Policy.ts
+++ b/src/libs/actions/Policy.ts
@@ -2498,12 +2498,30 @@ function deleteWorkspaceCategories(policyID: string, categoryNamesToDelete: stri
                 value: {
                     ...Object.keys(policyCategories).reduce<PolicyCategories>((acc, key) => {
                         if (categoryNamesToDelete.includes(key)) {
-                            return acc;
+                            acc[key] = {
+                                ...policyCategories[key],
+                                pendingAction: CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE,
+                                errors: null,
+                            };
                         }
 
-                        acc[key] = policyCategories[key];
                         return acc;
-                    }),
+                    }, {}),
+                },
+            },
+        ],
+        successData: [
+            {
+                onyxMethod: Onyx.METHOD.SET,
+                key: `${ONYXKEYS.COLLECTION.POLICY_CATEGORIES}${policyID}`,
+                value: {
+                    ...Object.keys(policyCategories).reduce<PolicyCategories>((acc, key) => {
+                        if (!categoryNamesToDelete.includes(key)) {
+                            acc[key] = policyCategories[key];
+                        }
+
+                        return acc;
+                    }, {}),
                 },
             },
         ],
@@ -2512,7 +2530,17 @@ function deleteWorkspaceCategories(policyID: string, categoryNamesToDelete: stri
                 onyxMethod: Onyx.METHOD.MERGE,
                 key: `${ONYXKEYS.COLLECTION.POLICY_CATEGORIES}${policyID}`,
                 value: {
-                    ...policyCategories,
+                    ...Object.keys(policyCategories).reduce<PolicyCategories>((acc, key) => {
+                        if (categoryNamesToDelete.includes(key)) {
+                            acc[key] = {
+                                ...policyCategories[key],
+                                pendingAction: null,
+                                errors: ErrorUtils.getMicroSecondOnyxError('workspace.categories.genericFailureMessage'),
+                            };
+                        }
+
+                        return acc;
+                    }, {}),
                 },
             },
         ],
diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index 3b3f2cc87f1c..d43f79660a62 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -19,6 +19,7 @@ import useLocalize from '@hooks/useLocalize';
 import useTheme from '@hooks/useTheme';
 import useThemeStyles from '@hooks/useThemeStyles';
 import useWindowDimensions from '@hooks/useWindowDimensions';
+import {deleteWorkspaceCategories, setWorkspaceCategoryEnabled} from '@libs/actions/Policy';
 import Navigation from '@libs/Navigation/Navigation';
 import type {CentralPaneNavigatorParamList} from '@navigation/types';
 import AdminPolicyAccessOrNotFoundWrapper from '@pages/workspace/AdminPolicyAccessOrNotFoundWrapper';
@@ -102,30 +103,58 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
 
     const renderButtons = useCallback(() => {
         if (selectedCategoriesArray.length > 0) {
-            const isAllEnabled = selectedCategoriesArray.every((category) => policyCategories?.[category].enabled);
+            const isAllEnabled = selectedCategoriesArray.every((categoryName) => policyCategories?.[categoryName].enabled);
 
             const options: Array<DropdownOption<DeepValueOf<typeof CONST.POLICY.CATEGORIES_BULK_ACTION_TYPES>>> = [
                 {
                     icon: Expensicons.Trashcan,
                     text: translate('workspace.categories.deleteCategories'),
                     value: CONST.POLICY.CATEGORIES_BULK_ACTION_TYPES.DELETE,
-                    onSelected: () => null,
+                    onSelected: () => {
+                        setSelectedCategories({});
+                        deleteWorkspaceCategories(route.params.policyID, selectedCategoriesArray);
+                    },
                 },
             ];
 
             if (isAllEnabled) {
+                const categoriesToDisable = selectedCategoriesArray
+                    .filter((categoryName) => policyCategories?.[categoryName].enabled)
+                    .reduce<Record<string, {name: string; enabled: boolean}>>((acc, categoryName) => {
+                        acc[categoryName] = {
+                            name: categoryName,
+                            enabled: false,
+                        };
+                        return acc;
+                    }, {});
+
                 options.push({
                     icon: Expensicons.Trashcan,
                     text: translate('workspace.categories.disableCategories'),
                     value: CONST.POLICY.CATEGORIES_BULK_ACTION_TYPES.DISABLE,
-                    onSelected: () => null,
+                    onSelected: () => {
+                        setSelectedCategories({});
+                        setWorkspaceCategoryEnabled(route.params.policyID, categoriesToDisable);
+                    },
                 });
             } else {
+                const categoriesToEnable = selectedCategoriesArray
+                    .filter((categoryName) => !policyCategories?.[categoryName].enabled)
+                    .reduce<Record<string, {name: string; enabled: boolean}>>((acc, categoryName) => {
+                        acc[categoryName] = {
+                            name: categoryName,
+                            enabled: true,
+                        };
+                        return acc;
+                    }, {});
                 options.push({
                     icon: Expensicons.Trashcan,
                     text: translate('workspace.categories.enableCategories'),
                     value: CONST.POLICY.CATEGORIES_BULK_ACTION_TYPES.ENABLE,
-                    onSelected: () => null,
+                    onSelected: () => {
+                        setSelectedCategories({});
+                        setWorkspaceCategoryEnabled(route.params.policyID, categoriesToEnable);
+                    },
                 });
             }
 

From bd2c8bf89ad1d4687e0fdfbd7221c29c67bcdc56 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Tue, 5 Mar 2024 15:05:21 +0100
Subject: [PATCH 05/32] feat: add red brick road pending action to categories
 menu item and categories list

---
 src/languages/en.ts                                    |  1 +
 src/languages/es.ts                                    |  1 +
 src/libs/Navigation/types.ts                           |  4 +++-
 src/libs/PolicyUtils.ts                                | 10 +++++++++-
 src/libs/actions/Policy.ts                             |  2 +-
 src/pages/workspace/WorkspaceInitialPage.tsx           | 10 +++++++++-
 .../workspace/categories/WorkspaceCategoriesPage.tsx   |  2 ++
 7 files changed, 26 insertions(+), 4 deletions(-)

diff --git a/src/languages/en.ts b/src/languages/en.ts
index 6a5c67fb0902..13be35410ce1 100755
--- a/src/languages/en.ts
+++ b/src/languages/en.ts
@@ -1765,6 +1765,7 @@ export default {
             deleteCategories: 'Delete categories',
             disableCategories: 'Disable categories',
             enableCategories: 'Enable categories',
+            deleteFailureMessage: 'An error occurred while deleting the category, please try again.',
             categoryName: 'Category name',
             requiresCategory: 'Members must categorize all spend',
             enableCategory: 'Enable category',
diff --git a/src/languages/es.ts b/src/languages/es.ts
index 7841846f465a..2c4eee57c2c1 100644
--- a/src/languages/es.ts
+++ b/src/languages/es.ts
@@ -1789,6 +1789,7 @@ export default {
             deleteCategories: 'Eliminar categorías',
             disableCategories: 'Desactivar categorías',
             enableCategories: 'Activar categorías',
+            deleteFailureMessage: 'Se ha producido un error al intentar eliminar la categoría. Por favor, inténtalo más tarde.',
             categoryName: 'Nombre de la categoría',
             requiresCategory: 'Los miembros deben categorizar todos los gastos',
             enableCategory: 'Activar categoría',
diff --git a/src/libs/Navigation/types.ts b/src/libs/Navigation/types.ts
index 6ffbb3a358b0..9776be8f1387 100644
--- a/src/libs/Navigation/types.ts
+++ b/src/libs/Navigation/types.ts
@@ -516,7 +516,9 @@ type FullScreenNavigatorParamList = {
 type BottomTabNavigatorParamList = {
     [SCREENS.HOME]: undefined;
     [SCREENS.ALL_SETTINGS]: undefined;
-    [SCREENS.WORKSPACE.INITIAL]: undefined;
+    [SCREENS.WORKSPACE.INITIAL]: {
+        policyID: string;
+    };
 };
 
 type SharedScreensParamList = {
diff --git a/src/libs/PolicyUtils.ts b/src/libs/PolicyUtils.ts
index 5b916148c6ee..2e9335ab040b 100644
--- a/src/libs/PolicyUtils.ts
+++ b/src/libs/PolicyUtils.ts
@@ -4,7 +4,7 @@ import type {ValueOf} from 'type-fest';
 import CONST from '@src/CONST';
 import ONYXKEYS from '@src/ONYXKEYS';
 import ROUTES from '@src/ROUTES';
-import type {PersonalDetailsList, Policy, PolicyMembers, PolicyTagList, PolicyTags} from '@src/types/onyx';
+import type {PersonalDetailsList, Policy, PolicyCategories, PolicyMembers, PolicyTagList, PolicyTags} from '@src/types/onyx';
 import type {EmptyObject} from '@src/types/utils/EmptyObject';
 import {isEmptyObject} from '@src/types/utils/EmptyObject';
 import Navigation from './Navigation/Navigation';
@@ -30,6 +30,13 @@ function hasPolicyMemberError(policyMembers: OnyxEntry<PolicyMembers>): boolean
     return Object.values(policyMembers ?? {}).some((member) => Object.keys(member?.errors ?? {}).length > 0);
 }
 
+/**
+ * Check if the policy has any errors within the categories.
+ */
+function hasPolicyCategoriesError(policyCategories: OnyxEntry<PolicyCategories>): boolean {
+    return Object.keys(policyCategories ?? {}).some((categoryName) => Object.keys(policyCategories?.[categoryName]?.errors ?? {}).length > 0);
+}
+
 /**
  * Check if the policy has any error fields.
  */
@@ -291,6 +298,7 @@ export {
     getPathWithoutPolicyID,
     getPolicyMembersByIdWithoutCurrentUser,
     goBackFromInvalidPolicy,
+    hasPolicyCategoriesError,
 };
 
 export type {MemberEmailsToAccountIDs};
diff --git a/src/libs/actions/Policy.ts b/src/libs/actions/Policy.ts
index 48b87a234b6a..998facc027e3 100644
--- a/src/libs/actions/Policy.ts
+++ b/src/libs/actions/Policy.ts
@@ -2535,7 +2535,7 @@ function deleteWorkspaceCategories(policyID: string, categoryNamesToDelete: stri
                             acc[key] = {
                                 ...policyCategories[key],
                                 pendingAction: null,
-                                errors: ErrorUtils.getMicroSecondOnyxError('workspace.categories.genericFailureMessage'),
+                                errors: ErrorUtils.getMicroSecondOnyxError('workspace.categories.deleteFailureMessage'),
                             };
                         }
 
diff --git a/src/pages/workspace/WorkspaceInitialPage.tsx b/src/pages/workspace/WorkspaceInitialPage.tsx
index a18d2667f1ea..056200b2846e 100644
--- a/src/pages/workspace/WorkspaceInitialPage.tsx
+++ b/src/pages/workspace/WorkspaceInitialPage.tsx
@@ -45,6 +45,9 @@ type WorkspaceMenuItem = {
 type WorkspaceInitialPageOnyxProps = {
     /** Bank account attached to free plan */
     reimbursementAccount: OnyxEntry<OnyxTypes.ReimbursementAccount>;
+
+    /** Collection of categories attached to a policy */
+    policyCategories: OnyxEntry<OnyxTypes.PolicyCategories>;
 };
 
 type WorkspaceInitialPageProps = WithPolicyAndFullscreenLoadingProps & WorkspaceInitialPageOnyxProps & StackScreenProps<BottomTabNavigatorParamList, typeof SCREENS.WORKSPACE.INITIAL>;
@@ -54,7 +57,7 @@ function dismissError(policyID: string) {
     Policy.removeWorkspace(policyID);
 }
 
-function WorkspaceInitialPage({policyDraft, policy: policyProp, policyMembers, reimbursementAccount}: WorkspaceInitialPageProps) {
+function WorkspaceInitialPage({policyDraft, policy: policyProp, policyMembers, reimbursementAccount, policyCategories}: WorkspaceInitialPageProps) {
     const styles = useThemeStyles();
     const policy = policyDraft?.id ? policyDraft : policyProp;
     const [isCurrencyModalOpen, setIsCurrencyModalOpen] = useState(false);
@@ -95,6 +98,7 @@ function WorkspaceInitialPage({policyDraft, policy: policyProp, policyMembers, r
     }, [policyID, policyName]);
 
     const hasMembersError = PolicyUtils.hasPolicyMemberError(policyMembers);
+    const hasPolicyCategoryError = PolicyUtils.hasPolicyCategoriesError(policyCategories);
     const hasGeneralSettingsError = !isEmptyObject(policy?.errorFields?.generalSettings ?? {}) || !isEmptyObject(policy?.errorFields?.avatar ?? {});
     const shouldShowProtectedItems = PolicyUtils.isPolicyAdmin(policy);
     const isPaidGroupPolicy = PolicyUtils.isPaidGroupPolicy(policy);
@@ -167,6 +171,7 @@ function WorkspaceInitialPage({policyDraft, policy: policyProp, policyMembers, r
             translationKey: 'workspace.common.categories',
             icon: Expensicons.Folder,
             action: singleExecution(waitForNavigate(() => Navigation.navigate(ROUTES.WORKSPACE_CATEGORIES.getRoute(policyID)))),
+            brickRoadIndicator: hasPolicyCategoryError ? CONST.BRICK_ROAD_INDICATOR_STATUS.ERROR : undefined,
             routeName: SCREENS.WORKSPACE.CATEGORIES,
         },
         {
@@ -275,5 +280,8 @@ export default withPolicyAndFullscreenLoading(
         reimbursementAccount: {
             key: ONYXKEYS.REIMBURSEMENT_ACCOUNT,
         },
+        policyCategories: {
+            key: ({route}) => `${ONYXKEYS.COLLECTION.POLICY_CATEGORIES}${route.params?.policyID ?? '0'}`,
+        },
     })(WorkspaceInitialPage),
 );
diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index d43f79660a62..3d88cfa604a9 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -61,6 +61,8 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
                 text: value.name,
                 keyForList: value.name,
                 isSelected: !!selectedCategories[value.name],
+                isDisabled: value.pendingAction === CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE,
+                errors: value.errors,
                 rightElement: (
                     <View style={styles.flexRow}>
                         <Text style={[styles.disabledText, styles.alignSelfCenter]}>{value.enabled ? translate('workspace.common.enabled') : translate('workspace.common.disabled')}</Text>

From 34c64b1c4070d0d276bf7f46f7585778a07a10da Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Tue, 5 Mar 2024 15:09:36 +0100
Subject: [PATCH 06/32] refactor(types): remove PolicyForList, use ListItem
 type instead

---
 .../categories/WorkspaceCategoriesPage.tsx    | 27 ++++++++-----------
 1 file changed, 11 insertions(+), 16 deletions(-)

diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index 3d88cfa604a9..c3e3611776f1 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -13,6 +13,7 @@ import * as Illustrations from '@components/Icon/Illustrations';
 import ScreenWrapper from '@components/ScreenWrapper';
 import SelectionList from '@components/SelectionList';
 import TableListItem from '@components/SelectionList/TableListItem';
+import type {ListItem} from '@components/SelectionList/types';
 import Text from '@components/Text';
 import WorkspaceEmptyStateSection from '@components/WorkspaceEmptyStateSection';
 import useLocalize from '@hooks/useLocalize';
@@ -31,13 +32,7 @@ import type SCREENS from '@src/SCREENS';
 import type * as OnyxTypes from '@src/types/onyx';
 import type DeepValueOf from '@src/types/utils/DeepValueOf';
 
-type PolicyForList = {
-    value: string;
-    text: string;
-    keyForList: string;
-    isSelected: boolean;
-    rightElement: React.ReactNode;
-};
+type PolicyOption = ListItem;
 
 type WorkspaceCategoriesOnyxProps = {
     /** Collection of categories attached to a policy */
@@ -54,15 +49,15 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
     const [selectedCategories, setSelectedCategories] = useState<Record<string, boolean>>({});
     const dropdownButtonRef = useRef(null);
 
-    const categoryList = useMemo<PolicyForList[]>(
+    const categoryList = useMemo(
         () =>
-            Object.values(policyCategories ?? {}).map((value) => ({
-                value: value.name,
+            Object.values(policyCategories ?? {}).map<PolicyOption>((value) => ({
                 text: value.name,
                 keyForList: value.name,
                 isSelected: !!selectedCategories[value.name],
                 isDisabled: value.pendingAction === CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE,
-                errors: value.errors,
+                pendingAction: value.pendingAction,
+                errors: value.errors ?? undefined,
                 rightElement: (
                     <View style={styles.flexRow}>
                         <Text style={[styles.disabledText, styles.alignSelfCenter]}>{value.enabled ? translate('workspace.common.enabled') : translate('workspace.common.disabled')}</Text>
@@ -78,16 +73,16 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
         [policyCategories, selectedCategories, styles.alignSelfCenter, styles.disabledText, styles.flexRow, styles.p1, styles.pl2, theme.icon, translate],
     );
 
-    const toggleCategory = (category: PolicyForList) => {
+    const toggleCategory = (category: PolicyOption) => {
         setSelectedCategories((prev) => ({
             ...prev,
-            [category.value]: !prev[category.value],
+            [category.keyForList]: !prev[category.keyForList],
         }));
     };
 
     const toggleAllCategories = () => {
-        const isAllSelected = categoryList.every((category) => !!selectedCategories[category.value]);
-        setSelectedCategories(isAllSelected ? {} : Object.fromEntries(categoryList.map((item) => [item.value, true])));
+        const isAllSelected = categoryList.every((category) => !!selectedCategories[category.keyForList]);
+        setSelectedCategories(isAllSelected ? {} : Object.fromEntries(categoryList.map((item) => [item.keyForList, true])));
     };
 
     const getCustomListHeader = () => (
@@ -97,7 +92,7 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
         </View>
     );
 
-    const navigateToCategorySettings = (category: PolicyForList) => {
+    const navigateToCategorySettings = (category: PolicyOption) => {
         Navigation.navigate(ROUTES.WORKSPACE_CATEGORY_SETTINGS.getRoute(route.params.policyID, category.text));
     };
 

From 7713c28dde558ab1d5d5497c4dee436e295da151 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Tue, 5 Mar 2024 15:20:19 +0100
Subject: [PATCH 07/32] feat: handle dismiss errors in categories list

---
 src/pages/workspace/categories/WorkspaceCategoriesPage.tsx | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index c3e3611776f1..939ffd16a48c 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -25,6 +25,7 @@ import Navigation from '@libs/Navigation/Navigation';
 import type {CentralPaneNavigatorParamList} from '@navigation/types';
 import AdminPolicyAccessOrNotFoundWrapper from '@pages/workspace/AdminPolicyAccessOrNotFoundWrapper';
 import PaidPolicyAccessOrNotFoundWrapper from '@pages/workspace/PaidPolicyAccessOrNotFoundWrapper';
+import * as Policy from '@userActions/Policy';
 import CONST from '@src/CONST';
 import ONYXKEYS from '@src/ONYXKEYS';
 import ROUTES from '@src/ROUTES';
@@ -55,7 +56,6 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
                 text: value.name,
                 keyForList: value.name,
                 isSelected: !!selectedCategories[value.name],
-                isDisabled: value.pendingAction === CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE,
                 pendingAction: value.pendingAction,
                 errors: value.errors ?? undefined,
                 rightElement: (
@@ -96,6 +96,10 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
         Navigation.navigate(ROUTES.WORKSPACE_CATEGORY_SETTINGS.getRoute(route.params.policyID, category.text));
     };
 
+    const dismissError = (item: PolicyOption) => {
+        Policy.clearCategoryErrors(route.params.policyID, item.keyForList);
+    };
+
     const selectedCategoriesArray = Object.keys(selectedCategories).filter((key) => selectedCategories[key]);
 
     const renderButtons = useCallback(() => {
@@ -213,6 +217,7 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
                             onSelectAll={toggleAllCategories}
                             showScrollIndicator
                             ListItem={TableListItem}
+                            onDismissError={dismissError}
                             customListHeader={getCustomListHeader()}
                             listHeaderWrapperStyle={[styles.ph9, styles.pv3, styles.pb5]}
                         />

From 34b15a0581c6d3de9a394d808ac74f2f654eee0e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Tue, 5 Mar 2024 16:10:07 +0100
Subject: [PATCH 08/32] fix: prevent removing categories from list if pending
 action is delete

---
 src/pages/workspace/categories/WorkspaceCategoriesPage.tsx | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index 939ffd16a48c..9a317f5f3d0d 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -56,7 +56,6 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
                 text: value.name,
                 keyForList: value.name,
                 isSelected: !!selectedCategories[value.name],
-                pendingAction: value.pendingAction,
                 errors: value.errors ?? undefined,
                 rightElement: (
                     <View style={styles.flexRow}>

From c0b6b924644891430e8065c5407f9e21927a81e2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Tue, 5 Mar 2024 16:42:44 +0100
Subject: [PATCH 09/32] fix: disabled right element styles

---
 .../categories/WorkspaceCategoriesPage.tsx    | 40 +++++++++++--------
 1 file changed, 23 insertions(+), 17 deletions(-)

diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index 9a317f5f3d0d..4e999a136d2d 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -52,24 +52,30 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
 
     const categoryList = useMemo(
         () =>
-            Object.values(policyCategories ?? {}).map<PolicyOption>((value) => ({
-                text: value.name,
-                keyForList: value.name,
-                isSelected: !!selectedCategories[value.name],
-                errors: value.errors ?? undefined,
-                rightElement: (
-                    <View style={styles.flexRow}>
-                        <Text style={[styles.disabledText, styles.alignSelfCenter]}>{value.enabled ? translate('workspace.common.enabled') : translate('workspace.common.disabled')}</Text>
-                        <View style={[styles.p1, styles.pl2]}>
-                            <Icon
-                                src={Expensicons.ArrowRight}
-                                fill={theme.icon}
-                            />
+            Object.values(policyCategories ?? {}).map<PolicyOption>((value) => {
+                const isDisabled = value.pendingAction === CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE;
+                return {
+                    text: value.name,
+                    keyForList: value.name,
+                    isSelected: !!selectedCategories[value.name],
+                    isDisabled,
+                    errors: value.errors ?? undefined,
+                    rightElement: (
+                        <View style={[styles.flexRow, isDisabled && styles.buttonOpacityDisabled]}>
+                            <Text style={[styles.disabledText, styles.alignSelfCenter]}>
+                                {value.enabled ? translate('workspace.common.enabled') : translate('workspace.common.disabled')}
+                            </Text>
+                            <View style={[styles.p1, styles.pl2]}>
+                                <Icon
+                                    src={Expensicons.ArrowRight}
+                                    fill={theme.icon}
+                                />
+                            </View>
                         </View>
-                    </View>
-                ),
-            })),
-        [policyCategories, selectedCategories, styles.alignSelfCenter, styles.disabledText, styles.flexRow, styles.p1, styles.pl2, theme.icon, translate],
+                    ),
+                };
+            }),
+        [policyCategories, selectedCategories, styles.alignSelfCenter, styles.buttonOpacityDisabled, styles.disabledText, styles.flexRow, styles.p1, styles.pl2, theme.icon, translate],
     );
 
     const toggleCategory = (category: PolicyOption) => {

From 920ee8318fc223f5b2aefe9e28f00b9f5e1d4991 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Wed, 6 Mar 2024 10:31:48 +0100
Subject: [PATCH 10/32] refactor: optimize optimistic data setting & params as
 a json

---
 .../DeleteWorkspaceCategoriesParams.ts        |  6 +++++-
 src/libs/actions/Policy.ts                    | 20 ++++++-------------
 2 files changed, 11 insertions(+), 15 deletions(-)

diff --git a/src/libs/API/parameters/DeleteWorkspaceCategoriesParams.ts b/src/libs/API/parameters/DeleteWorkspaceCategoriesParams.ts
index f82841acdeba..07a8103a9b06 100644
--- a/src/libs/API/parameters/DeleteWorkspaceCategoriesParams.ts
+++ b/src/libs/API/parameters/DeleteWorkspaceCategoriesParams.ts
@@ -1,6 +1,10 @@
 type DeleteWorkspaceCategoriesParams = {
     policyID: string;
-    categories: string[];
+    /**
+     * A JSON-encoded string representing an array of category names to be deleted.
+     * Each element in the array is a string that specifies the name of a category.
+     */
+    categories: string;
 };
 
 export default DeleteWorkspaceCategoriesParams;
diff --git a/src/libs/actions/Policy.ts b/src/libs/actions/Policy.ts
index 998facc027e3..b6a565239b92 100644
--- a/src/libs/actions/Policy.ts
+++ b/src/libs/actions/Policy.ts
@@ -47,6 +47,7 @@ import type {
     PersonalDetailsList,
     Policy,
     PolicyCategories,
+    PolicyCategory,
     PolicyMember,
     PolicyTagList,
     RecentlyUsedCategories,
@@ -2495,19 +2496,10 @@ function deleteWorkspaceCategories(policyID: string, categoryNamesToDelete: stri
             {
                 onyxMethod: Onyx.METHOD.MERGE,
                 key: `${ONYXKEYS.COLLECTION.POLICY_CATEGORIES}${policyID}`,
-                value: {
-                    ...Object.keys(policyCategories).reduce<PolicyCategories>((acc, key) => {
-                        if (categoryNamesToDelete.includes(key)) {
-                            acc[key] = {
-                                ...policyCategories[key],
-                                pendingAction: CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE,
-                                errors: null,
-                            };
-                        }
-
-                        return acc;
-                    }, {}),
-                },
+                value: categoryNamesToDelete.reduce<Record<string, Partial<PolicyCategory>>>((acc, categoryName) => {
+                    acc[categoryName] = {pendingAction: CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE};
+                    return acc;
+                }, {}),
             },
         ],
         successData: [
@@ -2548,7 +2540,7 @@ function deleteWorkspaceCategories(policyID: string, categoryNamesToDelete: stri
 
     const parameters = {
         policyID,
-        categories: categoryNamesToDelete,
+        categories: JSON.stringify(categoryNamesToDelete),
     };
 
     API.write('DeleteWorkspaceCategories', parameters, onyxData);

From f0ec68add394172056aa0bb1eb7c1852a7717ce7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Wed, 6 Mar 2024 10:32:03 +0100
Subject: [PATCH 11/32] fix: add pending action to list item

---
 src/pages/workspace/categories/WorkspaceCategoriesPage.tsx | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index 4e999a136d2d..2044adad1fba 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -59,6 +59,7 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
                     keyForList: value.name,
                     isSelected: !!selectedCategories[value.name],
                     isDisabled,
+                    pendingAction: value.pendingAction,
                     errors: value.errors ?? undefined,
                     rightElement: (
                         <View style={[styles.flexRow, isDisabled && styles.buttonOpacityDisabled]}>

From fd18d44c4b8be55419a9883fe10560d793fe4a31 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Wed, 6 Mar 2024 10:33:19 +0100
Subject: [PATCH 12/32] refactor: remove preading value

---
 src/libs/actions/Policy.ts | 36 ++++++++++++++++--------------------
 1 file changed, 16 insertions(+), 20 deletions(-)

diff --git a/src/libs/actions/Policy.ts b/src/libs/actions/Policy.ts
index b6a565239b92..22ca0b8ab283 100644
--- a/src/libs/actions/Policy.ts
+++ b/src/libs/actions/Policy.ts
@@ -2506,34 +2506,30 @@ function deleteWorkspaceCategories(policyID: string, categoryNamesToDelete: stri
             {
                 onyxMethod: Onyx.METHOD.SET,
                 key: `${ONYXKEYS.COLLECTION.POLICY_CATEGORIES}${policyID}`,
-                value: {
-                    ...Object.keys(policyCategories).reduce<PolicyCategories>((acc, key) => {
-                        if (!categoryNamesToDelete.includes(key)) {
-                            acc[key] = policyCategories[key];
-                        }
+                value: Object.keys(policyCategories).reduce<PolicyCategories>((acc, key) => {
+                    if (!categoryNamesToDelete.includes(key)) {
+                        acc[key] = policyCategories[key];
+                    }
 
-                        return acc;
-                    }, {}),
-                },
+                    return acc;
+                }, {}),
             },
         ],
         failureData: [
             {
                 onyxMethod: Onyx.METHOD.MERGE,
                 key: `${ONYXKEYS.COLLECTION.POLICY_CATEGORIES}${policyID}`,
-                value: {
-                    ...Object.keys(policyCategories).reduce<PolicyCategories>((acc, key) => {
-                        if (categoryNamesToDelete.includes(key)) {
-                            acc[key] = {
-                                ...policyCategories[key],
-                                pendingAction: null,
-                                errors: ErrorUtils.getMicroSecondOnyxError('workspace.categories.deleteFailureMessage'),
-                            };
-                        }
+                value: Object.keys(policyCategories).reduce<PolicyCategories>((acc, key) => {
+                    if (categoryNamesToDelete.includes(key)) {
+                        acc[key] = {
+                            ...policyCategories[key],
+                            pendingAction: null,
+                            errors: ErrorUtils.getMicroSecondOnyxError('workspace.categories.deleteFailureMessage'),
+                        };
+                    }
 
-                        return acc;
-                    }, {}),
-                },
+                    return acc;
+                }, {}),
             },
         ],
     };

From 14470d840799058194756fd3f707aad750d03086 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Wed, 6 Mar 2024 11:55:07 +0100
Subject: [PATCH 13/32] fix(icon): document for enabling and document slash for
 disabling categories

---
 assets/images/documentSlash.svg                            | 1 +
 src/components/Icon/Expensicons.ts                         | 2 ++
 src/pages/workspace/categories/WorkspaceCategoriesPage.tsx | 4 ++--
 3 files changed, 5 insertions(+), 2 deletions(-)
 create mode 100644 assets/images/documentSlash.svg

diff --git a/assets/images/documentSlash.svg b/assets/images/documentSlash.svg
new file mode 100644
index 000000000000..626adcdc53a8
--- /dev/null
+++ b/assets/images/documentSlash.svg
@@ -0,0 +1 @@
+<svg width="20" height="20" viewBox="0 0 20 20" fill="none" xmlns="http://www.w3.org/2000/svg"><path d="M18 13.9861V7.5C18 7.22386 17.7761 7 17.5 7H11.5C11.2239 7 11 6.77614 11 6.5V0.5C11 0.223858 10.7761 0 10.5 0H4C3.52923 0 3.09644 0.162656 2.75478 0.434835L18 13.9861Z"/><path d="M2 7.79165V18C2 19.1046 2.89543 20 4 20H15.7344L2 7.79165Z"/><path d="M13 0L18 5H13.5C13.2239 5 13 4.77614 13 4.5V0Z"/><path d="M1.66437 2.2526C1.25159 1.88568 0.619519 1.92286 0.252601 2.33565C-0.114317 2.74843 -0.0771359 3.3805 0.335647 3.74742L18.3356 19.7474C18.7484 20.1143 19.3805 20.0772 19.7474 19.6644C20.1143 19.2516 20.0772 18.6195 19.6644 18.2526L1.66437 2.2526Z"/></svg>
diff --git a/src/components/Icon/Expensicons.ts b/src/components/Icon/Expensicons.ts
index d9f46203a703..6be265b36bf5 100644
--- a/src/components/Icon/Expensicons.ts
+++ b/src/components/Icon/Expensicons.ts
@@ -43,6 +43,7 @@ import Connect from '@assets/images/connect.svg';
 import Copy from '@assets/images/copy.svg';
 import CreditCard from '@assets/images/creditcard.svg';
 import Document from '@assets/images/document.svg';
+import DocumentSlash from '@assets/images/documentSlash.svg';
 import DotIndicatorUnfilled from '@assets/images/dot-indicator-unfilled.svg';
 import DotIndicator from '@assets/images/dot-indicator.svg';
 import DownArrow from '@assets/images/down.svg';
@@ -194,6 +195,7 @@ export {
     CreditCard,
     DeletedRoomAvatar,
     Document,
+    DocumentSlash,
     DomainRoomAvatar,
     DotIndicator,
     DotIndicatorUnfilled,
diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index 87c103ec8700..3ed82b655dd7 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -148,7 +148,7 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
                     }, {});
 
                 options.push({
-                    icon: Expensicons.Trashcan,
+                    icon: Expensicons.DocumentSlash,
                     text: translate('workspace.categories.disableCategories'),
                     value: CONST.POLICY.CATEGORIES_BULK_ACTION_TYPES.DISABLE,
                     onSelected: () => {
@@ -167,7 +167,7 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
                         return acc;
                     }, {});
                 options.push({
-                    icon: Expensicons.Trashcan,
+                    icon: Expensicons.Document,
                     text: translate('workspace.categories.enableCategories'),
                     value: CONST.POLICY.CATEGORIES_BULK_ACTION_TYPES.ENABLE,
                     onSelected: () => {

From a429fa18f1f7f5608f0b1e0794eb767ecb6400ef Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Wed, 6 Mar 2024 12:06:03 +0100
Subject: [PATCH 14/32] fix(update categories): add pending action update when
 updating categories

---
 src/libs/actions/Policy.ts | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/libs/actions/Policy.ts b/src/libs/actions/Policy.ts
index 892e7c8b8469..d171a7221bbc 100644
--- a/src/libs/actions/Policy.ts
+++ b/src/libs/actions/Policy.ts
@@ -2379,6 +2379,7 @@ function setWorkspaceCategoryEnabled(policyID: string, categoriesToUpdate: Recor
                             pendingFields: {
                                 enabled: CONST.RED_BRICK_ROAD_PENDING_ACTION.UPDATE,
                             },
+                            pendingAction: CONST.RED_BRICK_ROAD_PENDING_ACTION.UPDATE,
                         };
 
                         return acc;
@@ -2399,6 +2400,7 @@ function setWorkspaceCategoryEnabled(policyID: string, categoriesToUpdate: Recor
                             pendingFields: {
                                 enabled: null,
                             },
+                            pendingAction: null,
                         };
 
                         return acc;
@@ -2419,6 +2421,7 @@ function setWorkspaceCategoryEnabled(policyID: string, categoriesToUpdate: Recor
                             pendingFields: {
                                 enabled: null,
                             },
+                            pendingAction: null,
                         };
 
                         return acc;

From 1b2c87e66018fb1530cc276a9e8008eac881851f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Wed, 6 Mar 2024 12:33:36 +0100
Subject: [PATCH 15/32] fix(styles): 50% width for small screen width in button
 with dropdown

---
 src/pages/workspace/categories/WorkspaceCategoriesPage.tsx | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index 3ed82b655dd7..4af50d2e9d69 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -186,6 +186,7 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
                     buttonSize={CONST.DROPDOWN_BUTTON_SIZE.MEDIUM}
                     customText={translate('workspace.common.selected', {selectedNumber: selectedCategoriesArray.length})}
                     options={options}
+                    style={[isSmallScreenWidth && styles.w50, isSmallScreenWidth && styles.mb3]}
                 />
             );
         }

From 790a0e6327734a18346a0e981f420543afac3016 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Wed, 6 Mar 2024 12:36:26 +0100
Subject: [PATCH 16/32] fix: fix icon fill color on mobile platforms

---
 assets/images/documentSlash.svg | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/assets/images/documentSlash.svg b/assets/images/documentSlash.svg
index 626adcdc53a8..ebb183142e40 100644
--- a/assets/images/documentSlash.svg
+++ b/assets/images/documentSlash.svg
@@ -1 +1 @@
-<svg width="20" height="20" viewBox="0 0 20 20" fill="none" xmlns="http://www.w3.org/2000/svg"><path d="M18 13.9861V7.5C18 7.22386 17.7761 7 17.5 7H11.5C11.2239 7 11 6.77614 11 6.5V0.5C11 0.223858 10.7761 0 10.5 0H4C3.52923 0 3.09644 0.162656 2.75478 0.434835L18 13.9861Z"/><path d="M2 7.79165V18C2 19.1046 2.89543 20 4 20H15.7344L2 7.79165Z"/><path d="M13 0L18 5H13.5C13.2239 5 13 4.77614 13 4.5V0Z"/><path d="M1.66437 2.2526C1.25159 1.88568 0.619519 1.92286 0.252601 2.33565C-0.114317 2.74843 -0.0771359 3.3805 0.335647 3.74742L18.3356 19.7474C18.7484 20.1143 19.3805 20.0772 19.7474 19.6644C20.1143 19.2516 20.0772 18.6195 19.6644 18.2526L1.66437 2.2526Z"/></svg>
+<svg width="20" height="20" viewBox="0 0 20 20" style="enable-background:new 0 0 20 20" xmlns="http://www.w3.org/2000/svg"><path d="M18 13.9861V7.5C18 7.22386 17.7761 7 17.5 7H11.5C11.2239 7 11 6.77614 11 6.5V0.5C11 0.223858 10.7761 0 10.5 0H4C3.52923 0 3.09644 0.162656 2.75478 0.434835L18 13.9861Z"/><path d="M2 7.79165V18C2 19.1046 2.89543 20 4 20H15.7344L2 7.79165Z"/><path d="M13 0L18 5H13.5C13.2239 5 13 4.77614 13 4.5V0Z"/><path d="M1.66437 2.2526C1.25159 1.88568 0.619519 1.92286 0.252601 2.33565C-0.114317 2.74843 -0.0771359 3.3805 0.335647 3.74742L18.3356 19.7474C18.7484 20.1143 19.3805 20.0772 19.7474 19.6644C20.1143 19.2516 20.0772 18.6195 19.6644 18.2526L1.66437 2.2526Z"/></svg>

From 14c12212ce70b7bdac5f06ef63cc909c0a3b3c8d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Wed, 6 Mar 2024 12:44:11 +0100
Subject: [PATCH 17/32] refactor: remove usecallback got getHeaderButtons

---
 .../categories/WorkspaceCategoriesPage.tsx        | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index 4af50d2e9d69..076e461f45ba 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -114,13 +114,17 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
         Navigation.navigate(ROUTES.WORKSPACE_CATEGORY_SETTINGS.getRoute(route.params.policyID, category.text));
     };
 
+    const navigateToCategoriesSettings = () => {
+        Navigation.navigate(ROUTES.WORKSPACE_CATEGORIES_SETTINGS.getRoute(route.params.policyID));
+    };
+
     const dismissError = (item: PolicyOption) => {
         Policy.clearCategoryErrors(route.params.policyID, item.keyForList);
     };
 
     const selectedCategoriesArray = Object.keys(selectedCategories).filter((key) => selectedCategories[key]);
 
-    const renderButtons = useCallback(() => {
+    const getHeaderButtons = () => {
         if (selectedCategoriesArray.length > 0) {
             const isAllEnabled = selectedCategoriesArray.every((categoryName) => policyCategories?.[categoryName].enabled);
 
@@ -190,9 +194,6 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
                 />
             );
         }
-        const navigateToCategoriesSettings = () => {
-            Navigation.navigate(ROUTES.WORKSPACE_CATEGORIES_SETTINGS.getRoute(route.params.policyID));
-        };
 
         return (
             <View style={[styles.w100, styles.flexRow, isSmallScreenWidth && styles.mb3]}>
@@ -205,7 +206,7 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
                 />
             </View>
         );
-    }, [isSmallScreenWidth, policyCategories, route.params.policyID, selectedCategoriesArray, styles.flexRow, styles.mb3, styles.w100, styles.w50, translate]);
+    };
 
     const isLoading = !isOffline && policyCategories === undefined;
 
@@ -223,9 +224,9 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
                         title={translate('workspace.common.categories')}
                         shouldShowBackButton={isSmallScreenWidth}
                     >
-                        {!isSmallScreenWidth && renderButtons()}
+                        {!isSmallScreenWidth && getHeaderButtons()}
                     </HeaderWithBackButton>
-                    {isSmallScreenWidth && <View style={[styles.pl5, styles.pr5]}>{renderButtons()}</View>}
+                    {isSmallScreenWidth && <View style={[styles.pl5, styles.pr5]}>{getHeaderButtons()}</View>}
                     <View style={[styles.ph5, styles.pb5]}>
                         <Text style={[styles.textNormal, styles.colorMuted]}>{translate('workspace.categories.subtitle')}</Text>
                     </View>

From c26e2a7feedcb57313fea8e25c8a999026600181 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Wed, 6 Mar 2024 12:44:34 +0100
Subject: [PATCH 18/32] fix: eslint issues

---
 src/pages/workspace/categories/WorkspaceCategoriesPage.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index 076e461f45ba..b9c0d2e8cbbd 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -1,5 +1,5 @@
 import type {StackScreenProps} from '@react-navigation/stack';
-import React, {useCallback, useEffect, useMemo, useRef, useState} from 'react';
+import React, {useEffect, useMemo, useRef, useState} from 'react';
 import {ActivityIndicator, View} from 'react-native';
 import {withOnyx} from 'react-native-onyx';
 import type {OnyxEntry} from 'react-native-onyx';

From d06f58eda09bcec8175a3353956499c2a0387904 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Wed, 6 Mar 2024 12:58:43 +0100
Subject: [PATCH 19/32] fix: prevent selecting hidden pending removal
 categories

---
 src/pages/workspace/categories/WorkspaceCategoriesPage.tsx | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index b9c0d2e8cbbd..b0e354cd45d8 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -100,7 +100,11 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
 
     const toggleAllCategories = () => {
         const isAllSelected = categoryList.every((category) => !!selectedCategories[category.keyForList]);
-        setSelectedCategories(isAllSelected ? {} : Object.fromEntries(categoryList.map((item) => [item.keyForList, true])));
+        setSelectedCategories(
+            isAllSelected
+                ? {}
+                : Object.fromEntries(categoryList.filter((category) => category.pendingAction !== CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE).map((item) => [item.keyForList, true])),
+        );
     };
 
     const getCustomListHeader = () => (

From 05ee7390fad6d65880f9947e01e89559d1d0853c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Wed, 6 Mar 2024 13:50:24 +0100
Subject: [PATCH 20/32] refactor(ts): remove generic from
 ButtonWithDropdownMenu

---
 src/pages/workspace/categories/WorkspaceCategoriesPage.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index b0e354cd45d8..dbc26b6427df 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -186,7 +186,7 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
             }
 
             return (
-                <ButtonWithDropdownMenu<DeepValueOf<typeof CONST.POLICY.CATEGORIES_BULK_ACTION_TYPES>>
+                <ButtonWithDropdownMenu
                     buttonRef={dropdownButtonRef}
                     onPress={() => null}
                     shouldAlwaysShowDropdownMenu

From 9c1aeb63829c368d26994e3a1703d4edaaf686c2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Wed, 6 Mar 2024 13:54:23 +0100
Subject: [PATCH 21/32] refactor(onyx actions): use const as api command name
 instead of string

---
 src/libs/actions/Policy.ts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/libs/actions/Policy.ts b/src/libs/actions/Policy.ts
index d171a7221bbc..507a905b93a6 100644
--- a/src/libs/actions/Policy.ts
+++ b/src/libs/actions/Policy.ts
@@ -2436,7 +2436,7 @@ function setWorkspaceCategoryEnabled(policyID: string, categoriesToUpdate: Recor
         categories: JSON.stringify(Object.keys(categoriesToUpdate).map((key) => categoriesToUpdate[key])),
     };
 
-    API.write('SetWorkspaceCategoriesEnabled', parameters, onyxData);
+    API.write(WRITE_COMMANDS.SET_WORKSPACE_CATEGORIES_ENABLED, parameters, onyxData);
 }
 
 function setWorkspaceRequiresCategory(policyID: string, requiresCategory: boolean) {
@@ -2490,7 +2490,7 @@ function setWorkspaceRequiresCategory(policyID: string, requiresCategory: boolea
         requiresCategory,
     };
 
-    API.write('SetWorkspaceRequiresCategory', parameters, onyxData);
+    API.write(WRITE_COMMANDS.SET_WORKSPACE_REQUIRES_CATEGORY, parameters, onyxData);
 }
 
 function clearCategoryErrors(policyID: string, categoryName: string) {
@@ -2558,7 +2558,7 @@ function deleteWorkspaceCategories(policyID: string, categoryNamesToDelete: stri
         categories: JSON.stringify(categoryNamesToDelete),
     };
 
-    API.write('DeleteWorkspaceCategories', parameters, onyxData);
+    API.write(WRITE_COMMANDS.DELETE_WORKSPACE_CATEGORIES, parameters, onyxData);
 }
 
 export {

From 3a7b40e4e4529682bdcec7cbc8d6411fa9997010 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Wed, 6 Mar 2024 13:55:33 +0100
Subject: [PATCH 22/32] refactor(ts): move generic from map to useMemo

---
 src/pages/workspace/categories/WorkspaceCategoriesPage.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index dbc26b6427df..560eda310efa 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -62,9 +62,9 @@ function WorkspaceCategoriesPage({policyCategories, route}: WorkspaceCategoriesP
         // eslint-disable-next-line react-hooks/exhaustive-deps
     }, []);
 
-    const categoryList = useMemo(
+    const categoryList = useMemo<PolicyOption[]>(
         () =>
-            Object.values(policyCategories ?? {}).map<PolicyOption>((value) => {
+            Object.values(policyCategories ?? {}).map((value) => {
                 const isDisabled = value.pendingAction === CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE;
                 return {
                     text: value.name,

From 70a96d506cb2d946fe60a18e7d56c80c9ed6e7d1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Wed, 6 Mar 2024 14:02:04 +0100
Subject: [PATCH 23/32] fix: rename svg as per convention

---
 assets/images/{documentSlash.svg => document-slash.svg} | 0
 src/components/Icon/Expensicons.ts                      | 2 +-
 2 files changed, 1 insertion(+), 1 deletion(-)
 rename assets/images/{documentSlash.svg => document-slash.svg} (100%)

diff --git a/assets/images/documentSlash.svg b/assets/images/document-slash.svg
similarity index 100%
rename from assets/images/documentSlash.svg
rename to assets/images/document-slash.svg
diff --git a/src/components/Icon/Expensicons.ts b/src/components/Icon/Expensicons.ts
index 6be265b36bf5..3f1cde92a583 100644
--- a/src/components/Icon/Expensicons.ts
+++ b/src/components/Icon/Expensicons.ts
@@ -42,8 +42,8 @@ import Concierge from '@assets/images/concierge.svg';
 import Connect from '@assets/images/connect.svg';
 import Copy from '@assets/images/copy.svg';
 import CreditCard from '@assets/images/creditcard.svg';
+import DocumentSlash from '@assets/images/document-slash.svg';
 import Document from '@assets/images/document.svg';
-import DocumentSlash from '@assets/images/documentSlash.svg';
 import DotIndicatorUnfilled from '@assets/images/dot-indicator-unfilled.svg';
 import DotIndicator from '@assets/images/dot-indicator.svg';
 import DownArrow from '@assets/images/down.svg';

From f57b89301e20e8b41531f396b6bcb675112ae7a3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Thu, 7 Mar 2024 13:07:03 +0100
Subject: [PATCH 24/32] fix: remove pendingAction from update category

---
 src/libs/actions/Policy.ts                                 | 3 ---
 src/pages/workspace/categories/WorkspaceCategoriesPage.tsx | 2 +-
 2 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/src/libs/actions/Policy.ts b/src/libs/actions/Policy.ts
index b24e530f58ed..3932fb98ff37 100644
--- a/src/libs/actions/Policy.ts
+++ b/src/libs/actions/Policy.ts
@@ -2379,7 +2379,6 @@ function setWorkspaceCategoryEnabled(policyID: string, categoriesToUpdate: Recor
                             pendingFields: {
                                 enabled: CONST.RED_BRICK_ROAD_PENDING_ACTION.UPDATE,
                             },
-                            pendingAction: CONST.RED_BRICK_ROAD_PENDING_ACTION.UPDATE,
                         };
 
                         return acc;
@@ -2400,7 +2399,6 @@ function setWorkspaceCategoryEnabled(policyID: string, categoriesToUpdate: Recor
                             pendingFields: {
                                 enabled: null,
                             },
-                            pendingAction: null,
                         };
 
                         return acc;
@@ -2421,7 +2419,6 @@ function setWorkspaceCategoryEnabled(policyID: string, categoriesToUpdate: Recor
                             pendingFields: {
                                 enabled: null,
                             },
-                            pendingAction: null,
                         };
 
                         return acc;
diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index a7729e0141a3..b61cbc0de458 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -72,7 +72,7 @@ function WorkspaceCategoriesPage({policy, policyCategories, route}: WorkspaceCat
             Object.values(policyCategories ?? {})
                 .sort((a, b) => localeCompare(a.name, b.name))
                 .map((value) => {
-                    const isDisabled = value.pendingAction === CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE;
+                    const isDisabled = value.pendingAction === CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE || Object.values(value.pendingFields ?? {}).length > 0;
                     return {
                         text: value.name,
                         keyForList: value.name,

From 594f1e550a53d4d8613b9b802d6ab22258cd3eb0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Thu, 7 Mar 2024 13:41:17 +0100
Subject: [PATCH 25/32] fix: iterate using categoryNames in failure data

---
 src/libs/actions/Policy.ts | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/src/libs/actions/Policy.ts b/src/libs/actions/Policy.ts
index 3932fb98ff37..7c8c2bcd62f6 100644
--- a/src/libs/actions/Policy.ts
+++ b/src/libs/actions/Policy.ts
@@ -2585,15 +2585,11 @@ function deleteWorkspaceCategories(policyID: string, categoryNamesToDelete: stri
             {
                 onyxMethod: Onyx.METHOD.MERGE,
                 key: `${ONYXKEYS.COLLECTION.POLICY_CATEGORIES}${policyID}`,
-                value: Object.keys(policyCategories).reduce<PolicyCategories>((acc, key) => {
-                    if (categoryNamesToDelete.includes(key)) {
-                        acc[key] = {
-                            ...policyCategories[key],
-                            pendingAction: null,
-                            errors: ErrorUtils.getMicroSecondOnyxError('workspace.categories.deleteFailureMessage'),
-                        };
-                    }
-
+                value: categoryNamesToDelete.reduce<Record<string, Partial<PolicyCategory>>>((acc, categoryName) => {
+                    acc[categoryName] = {
+                        pendingAction: null,
+                        errors: ErrorUtils.getMicroSecondOnyxError('workspace.categories.deleteFailureMessage'),
+                    };
                     return acc;
                 }, {}),
             },

From 25f45f6754b85ef73987b1f64e453a3c8db19ce5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Thu, 7 Mar 2024 13:50:33 +0100
Subject: [PATCH 26/32] refactor: iterate using categoryNames in success data

---
 src/libs/actions/Policy.ts | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/src/libs/actions/Policy.ts b/src/libs/actions/Policy.ts
index 7c8c2bcd62f6..b61889f738d5 100644
--- a/src/libs/actions/Policy.ts
+++ b/src/libs/actions/Policy.ts
@@ -2555,8 +2555,6 @@ function clearCategoryErrors(policyID: string, categoryName: string) {
 }
 
 function deleteWorkspaceCategories(policyID: string, categoryNamesToDelete: string[]) {
-    const policyCategories = allPolicyCategories?.[`${ONYXKEYS.COLLECTION.POLICY_CATEGORIES}${policyID}`] ?? {};
-
     const onyxData: OnyxData = {
         optimisticData: [
             {
@@ -2570,13 +2568,10 @@ function deleteWorkspaceCategories(policyID: string, categoryNamesToDelete: stri
         ],
         successData: [
             {
-                onyxMethod: Onyx.METHOD.SET,
+                onyxMethod: Onyx.METHOD.MERGE,
                 key: `${ONYXKEYS.COLLECTION.POLICY_CATEGORIES}${policyID}`,
-                value: Object.keys(policyCategories).reduce<PolicyCategories>((acc, key) => {
-                    if (!categoryNamesToDelete.includes(key)) {
-                        acc[key] = policyCategories[key];
-                    }
-
+                value: categoryNamesToDelete.reduce<Record<string, null>>((acc, categoryName) => {
+                    acc[categoryName] = null;
                     return acc;
                 }, {}),
             },

From f4fbbe76ca3b69634cfb7fd3d04a4944c35e87c6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Thu, 7 Mar 2024 14:26:53 +0100
Subject: [PATCH 27/32] refactor: toggle categories

---
 .../categories/WorkspaceCategoriesPage.tsx    | 20 +++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index b61cbc0de458..1072d2b5cf87 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -111,19 +111,19 @@ function WorkspaceCategoriesPage({policy, policyCategories, route}: WorkspaceCat
     );
 
     const toggleCategory = (category: PolicyOption) => {
-        setSelectedCategories((prev) => ({
-            ...prev,
-            [category.keyForList]: !prev[category.keyForList],
-        }));
+        setSelectedCategories((prev) => {
+            if (prev[category.keyForList]) {
+                const {[category.keyForList]: omittedCategory, ...newCategories} = prev;
+                return newCategories;
+            }
+            return {...prev, [category.keyForList]: true};
+        });
     };
 
     const toggleAllCategories = () => {
-        const isAllSelected = categoryList.every((category) => !!selectedCategories[category.keyForList]);
-        setSelectedCategories(
-            isAllSelected
-                ? {}
-                : Object.fromEntries(categoryList.filter((category) => category.pendingAction !== CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE).map((item) => [item.keyForList, true])),
-        );
+        const availableCategories = categoryList.filter((category) => category.pendingAction !== CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE);
+        const isAllSelected = availableCategories.length === Object.keys(selectedCategories).length;
+        setSelectedCategories(isAllSelected ? {} : Object.fromEntries(availableCategories.map((item) => [item.keyForList, true])));
     };
 
     const getCustomListHeader = () => (

From 44b019c1054cdfdaf5329d1d96b46bd7f7067c4a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Fri, 8 Mar 2024 10:54:22 +0100
Subject: [PATCH 28/32] fix(categories selection): include disable and enable
 options when selecting mix states

---
 .../categories/WorkspaceCategoriesPage.tsx    | 30 ++++++++++---------
 1 file changed, 16 insertions(+), 14 deletions(-)

diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index 1072d2b5cf87..eec2b4831e10 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -152,22 +152,21 @@ function WorkspaceCategoriesPage({policy, policyCategories, route}: WorkspaceCat
     const selectedCategoriesArray = Object.keys(selectedCategories).filter((key) => selectedCategories[key]);
 
     const getHeaderButtons = () => {
-        if (selectedCategoriesArray.length > 0) {
-            const isAllEnabled = selectedCategoriesArray.every((categoryName) => policyCategories?.[categoryName].enabled);
+        const options: Array<DropdownOption<DeepValueOf<typeof CONST.POLICY.CATEGORIES_BULK_ACTION_TYPES>>> = [];
 
-            const options: Array<DropdownOption<DeepValueOf<typeof CONST.POLICY.CATEGORIES_BULK_ACTION_TYPES>>> = [
-                {
-                    icon: Expensicons.Trashcan,
-                    text: translate('workspace.categories.deleteCategories'),
-                    value: CONST.POLICY.CATEGORIES_BULK_ACTION_TYPES.DELETE,
-                    onSelected: () => {
-                        setSelectedCategories({});
-                        deleteWorkspaceCategories(route.params.policyID, selectedCategoriesArray);
-                    },
+        if (selectedCategoriesArray.length > 0) {
+            options.push({
+                icon: Expensicons.Trashcan,
+                text: translate('workspace.categories.deleteCategories'),
+                value: CONST.POLICY.CATEGORIES_BULK_ACTION_TYPES.DELETE,
+                onSelected: () => {
+                    setSelectedCategories({});
+                    deleteWorkspaceCategories(route.params.policyID, selectedCategoriesArray);
                 },
-            ];
+            });
 
-            if (isAllEnabled) {
+            const enabledCategories = selectedCategoriesArray.filter((categoryName) => policyCategories?.[categoryName].enabled);
+            if (enabledCategories.length > 0) {
                 const categoriesToDisable = selectedCategoriesArray
                     .filter((categoryName) => policyCategories?.[categoryName].enabled)
                     .reduce<Record<string, {name: string; enabled: boolean}>>((acc, categoryName) => {
@@ -187,7 +186,10 @@ function WorkspaceCategoriesPage({policy, policyCategories, route}: WorkspaceCat
                         setWorkspaceCategoryEnabled(route.params.policyID, categoriesToDisable);
                     },
                 });
-            } else {
+            }
+
+            const disabledCategories = selectedCategoriesArray.filter((categoryName) => !policyCategories?.[categoryName].enabled);
+            if (disabledCategories.length > 0) {
                 const categoriesToEnable = selectedCategoriesArray
                     .filter((categoryName) => !policyCategories?.[categoryName].enabled)
                     .reduce<Record<string, {name: string; enabled: boolean}>>((acc, categoryName) => {

From 4e6f2440cd1dd8aa13123881da00e96d76d56e61 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Mon, 11 Mar 2024 10:20:08 +0100
Subject: [PATCH 29/32] fix: conflicts

---
 src/languages/en.ts | 1 -
 src/languages/es.ts | 1 -
 2 files changed, 2 deletions(-)

diff --git a/src/languages/en.ts b/src/languages/en.ts
index 08aa1784d9b4..1be1f436548a 100755
--- a/src/languages/en.ts
+++ b/src/languages/en.ts
@@ -1757,7 +1757,6 @@ export default {
             mustBeOnlineToViewMembers: 'You must be online in order to view members of this workspace.',
             requested: 'Requested',
             distanceRates: 'Distance rates',
-            selected: ({selectedNumber}) => `${selectedNumber} selected`,
         },
         type: {
             free: 'Free',
diff --git a/src/languages/es.ts b/src/languages/es.ts
index c439ef0d3e10..e42d90a118c0 100644
--- a/src/languages/es.ts
+++ b/src/languages/es.ts
@@ -1781,7 +1781,6 @@ export default {
             mustBeOnlineToViewMembers: 'Debes estar en línea para poder ver los miembros de este espacio de trabajo.',
             requested: 'Solicitado',
             distanceRates: 'Tasas de distancia',
-            selected: ({selectedNumber}) => `${selectedNumber} seleccionados`,
         },
         type: {
             free: 'Gratis',

From cbdfe081eb0292b90fba5703f273f59ebaded2a7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Tue, 12 Mar 2024 15:15:44 +0100
Subject: [PATCH 30/32] fix: ts issues

---
 src/pages/workspace/categories/WorkspaceCategoriesPage.tsx | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
index b85076e4ae61..03bc3438fe40 100644
--- a/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
+++ b/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx
@@ -36,7 +36,10 @@ import type SCREENS from '@src/SCREENS';
 import type * as OnyxTypes from '@src/types/onyx';
 import type DeepValueOf from '@src/types/utils/DeepValueOf';
 
-type PolicyOption = ListItem;
+type PolicyOption = ListItem & {
+    /** Category name is used as a key for the selectedCategories state */
+    keyForList: string;
+};
 
 type WorkspaceCategoriesOnyxProps = {
     /** The policy the user is accessing. */
@@ -134,7 +137,7 @@ function WorkspaceCategoriesPage({policy, policyCategories, route}: WorkspaceCat
     );
 
     const navigateToCategorySettings = (category: PolicyOption) => {
-        Navigation.navigate(ROUTES.WORKSPACE_CATEGORY_SETTINGS.getRoute(route.params.policyID, category.text));
+        Navigation.navigate(ROUTES.WORKSPACE_CATEGORY_SETTINGS.getRoute(route.params.policyID, category.keyForList));
     };
 
     const navigateToCategoriesSettings = () => {

From acacbb2133aafe8d77ba7bc09722a4546d7b9564 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Tue, 12 Mar 2024 15:27:05 +0100
Subject: [PATCH 31/32] fix: conflicts - restore actions

---
 src/libs/actions/Policy.ts | 103 +++++++++++++++++++++++++++++++++++++
 1 file changed, 103 insertions(+)

diff --git a/src/libs/actions/Policy.ts b/src/libs/actions/Policy.ts
index 71cd87b6d752..fe840beb7256 100644
--- a/src/libs/actions/Policy.ts
+++ b/src/libs/actions/Policy.ts
@@ -3180,6 +3180,107 @@ function enablePolicyWorkflows(policyID: string, enabled: boolean) {
     }
 }
 
+function renamePolicyTaglist(policyID: string, policyTagListName: {oldName: string; newName: string}, policyTags: OnyxEntry<PolicyTagList>) {
+    const newName = policyTagListName.newName;
+    const oldName = policyTagListName.oldName;
+    const oldPolicyTags = policyTags?.[oldName] ?? {};
+    const onyxData: OnyxData = {
+        optimisticData: [
+            {
+                onyxMethod: Onyx.METHOD.MERGE,
+                key: `${ONYXKEYS.COLLECTION.POLICY_TAGS}${policyID}`,
+                value: {
+                    [newName]: {...oldPolicyTags, name: newName, pendingAction: CONST.RED_BRICK_ROAD_PENDING_ACTION.ADD},
+                    [oldName]: null,
+                },
+            },
+        ],
+        successData: [
+            {
+                onyxMethod: Onyx.METHOD.MERGE,
+                key: `${ONYXKEYS.COLLECTION.POLICY_TAGS}${policyID}`,
+                value: {
+                    [newName]: {pendingAction: null},
+                    [oldName]: null,
+                },
+            },
+        ],
+        failureData: [
+            {
+                onyxMethod: Onyx.METHOD.MERGE,
+                key: `${ONYXKEYS.COLLECTION.POLICY_TAGS}${policyID}`,
+                value: {
+                    errors: {
+                        [oldName]: oldName,
+                        [newName]: ErrorUtils.getMicroSecondOnyxError('workspace.tags.genericFailureMessage'),
+                    },
+                    [newName]: null,
+                    [oldName]: oldPolicyTags,
+                },
+            },
+        ],
+    };
+    const parameters = {
+        policyID,
+        oldName,
+        newName,
+    };
+
+    API.write(WRITE_COMMANDS.RENAME_POLICY_TAG_LIST, parameters, onyxData);
+}
+
+function setPolicyRequiresTag(policyID: string, requiresTag: boolean) {
+    const onyxData: OnyxData = {
+        optimisticData: [
+            {
+                onyxMethod: Onyx.METHOD.MERGE,
+                key: `${ONYXKEYS.COLLECTION.POLICY}${policyID}`,
+                value: {
+                    requiresTag,
+                    errors: {requiresTag: null},
+                    pendingFields: {
+                        requiresTag: CONST.RED_BRICK_ROAD_PENDING_ACTION.UPDATE,
+                    },
+                },
+            },
+        ],
+        successData: [
+            {
+                onyxMethod: Onyx.METHOD.MERGE,
+                key: `${ONYXKEYS.COLLECTION.POLICY}${policyID}`,
+                value: {
+                    errors: {
+                        requiresTag: null,
+                    },
+                    pendingFields: {
+                        requiresTag: null,
+                    },
+                },
+            },
+        ],
+        failureData: [
+            {
+                onyxMethod: Onyx.METHOD.MERGE,
+                key: `${ONYXKEYS.COLLECTION.POLICY}${policyID}`,
+                value: {
+                    requiresTag: !requiresTag,
+                    errors: ErrorUtils.getMicroSecondOnyxError('workspace.tags.genericFailureMessage'),
+                    pendingFields: {
+                        requiresTag: null,
+                    },
+                },
+            },
+        ],
+    };
+
+    const parameters = {
+        policyID,
+        requiresTag,
+    };
+
+    API.write(WRITE_COMMANDS.SET_POLICY_REQUIRES_TAG, parameters, onyxData);
+}
+
 export {
     removeMembers,
     updateWorkspaceMembersRole,
@@ -3244,4 +3345,6 @@ export {
     enablePolicyTaxes,
     enablePolicyWorkflows,
     openPolicyDistanceRatesPage,
+    renamePolicyTaglist,
+    setPolicyRequiresTag,
 };

From 0770d27e53206b787ddfc9019484c6fd2e457656 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Chrab=C4=85szczewski?= <Arkadiuszchr@gmail.com>
Date: Wed, 13 Mar 2024 10:59:54 +0100
Subject: [PATCH 32/32] fix: conflicts, export deleteWorkspaceCategories

---
 src/libs/actions/Policy.ts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/libs/actions/Policy.ts b/src/libs/actions/Policy.ts
index c4fa95bb1922..68ed15ece9a9 100644
--- a/src/libs/actions/Policy.ts
+++ b/src/libs/actions/Policy.ts
@@ -3563,4 +3563,5 @@ export {
     openPolicyDistanceRatesPage,
     createPolicyTag,
     clearWorkspaceReimbursementErrors,
+    deleteWorkspaceCategories,
 };