Merge pull request #54105 from bernhardoj/fix/53931-member-added-offl…

…ine-doesnt-show-on-approval-flow Fix new member added while offline doesn't show in approval flow list
Expensify · Dec 20, 2024 · 006a1df · 006a1df
2 parents 01c0ad9 + d3e2dfc
commit 006a1df
Show file tree

Hide file tree

Showing 2 changed files with 43 additions and 1 deletion.
diff --git a/src/libs/actions/Policy/Member.ts b/src/libs/actions/Policy/Member.ts
@@ -19,6 +19,7 @@ import enhanceParameters from '@libs/Network/enhanceParameters';
 import Parser from '@libs/Parser';
 import * as PersonalDetailsUtils from '@libs/PersonalDetailsUtils';
 import * as PhoneNumber from '@libs/PhoneNumber';
+import {getDefaultApprover} from '@libs/PolicyUtils';
 import * as ReportActionsUtils from '@libs/ReportActionsUtils';
 import * as ReportUtils from '@libs/ReportUtils';
 import CONST from '@src/CONST';
@@ -637,7 +638,12 @@ function addMembersToWorkspace(invitedEmailsToAccountIDs: InvitedEmailsToAccount
     const successMembersState: OnyxCollectionInputValue<PolicyEmployee> = {};
     const failureMembersState: OnyxCollectionInputValue<PolicyEmployee> = {};
     logins.forEach((email) => {
-        optimisticMembersState[email] = {pendingAction: CONST.RED_BRICK_ROAD_PENDING_ACTION.ADD, role: CONST.POLICY.ROLE.USER};
+        optimisticMembersState[email] = {
+            email,
+            pendingAction: CONST.RED_BRICK_ROAD_PENDING_ACTION.ADD,
+            role: CONST.POLICY.ROLE.USER,
+            submitsTo: getDefaultApprover(allPolicies?.[policyKey]),
+        };
         successMembersState[email] = {pendingAction: null};
         failureMembersState[email] = {
             errors: ErrorUtils.getMicroSecondOnyxErrorWithTranslationKey('workspace.people.error.genericAdd'),

diff --git a/tests/actions/PolicyMemberTest.ts b/tests/actions/PolicyMemberTest.ts
@@ -228,4 +228,40 @@ describe('actions/PolicyMember', () => {
             });
         });
     });
+
+    describe('addMembersToWorkspace', () => {
+        it('Add a new member to a workspace', async () => {
+            const policyID = '1';
+            const defaultApprover = '[email protected]';
+            const newUserEmail = '[email protected]';
+
+            await Onyx.set(`${ONYXKEYS.COLLECTION.POLICY}${policyID}`, {
+                ...createRandomPolicy(Number(policyID)),
+                approver: defaultApprover,
+            });
+
+            mockFetch?.pause?.();
+            Member.addMembersToWorkspace({[newUserEmail]: 1234}, 'Welcome', policyID, []);
+
+            await waitForBatchedUpdates();
+
+            await new Promise<void>((resolve) => {
+                const connection = Onyx.connect({
+                    key: `${ONYXKEYS.COLLECTION.POLICY}${policyID}`,
+                    waitForCollectionCallback: false,
+                    callback: (policy) => {
+                        Onyx.disconnect(connection);
+                        const newEmployee = policy?.employeeList?.[newUserEmail];
+                        expect(newEmployee).not.toBeUndefined();
+                        expect(newEmployee?.email).toBe(newUserEmail);
+                        expect(newEmployee?.pendingAction).toBe(CONST.RED_BRICK_ROAD_PENDING_ACTION.ADD);
+                        expect(newEmployee?.role).toBe(CONST.POLICY.ROLE.USER);
+                        expect(newEmployee?.submitsTo).toBe(defaultApprover);
+                        resolve();
+                    },
+                });
+            });
+            await mockFetch?.resume?.();
+        });
+    });
 });