Merge pull request #37948 from Expensify/cmartins-fixMemberSelection

Fix free policy member selection
Expensify · Mar 8, 2024 · 86a9904 · 86a9904
2 parents 9132e05 + 9941f77
commit 86a9904
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/src/pages/workspace/WorkspaceMembersPage.tsx b/src/pages/workspace/WorkspaceMembersPage.tsx
@@ -257,15 +257,11 @@ function WorkspaceMembersPage({policyMembers, personalDetails, route, policy, se
     /** Opens the member details page */
     const openMemberDetails = useCallback(
         (item: MemberOption) => {
-            if (!isPolicyAdmin) {
+            if (!isPolicyAdmin || !PolicyUtils.isPaidGroupPolicy(policy)) {
                 Navigation.navigate(ROUTES.PROFILE.getRoute(item.accountID));
                 return;
             }
 
-            if (!PolicyUtils.isPaidGroupPolicy(policy)) {
-                return;
-            }
-
             Navigation.navigate(ROUTES.WORKSPACE_MEMBER_DETAILS.getRoute(route.params.policyID, item.accountID, Navigation.getActiveRoute()));
         },
         [isPolicyAdmin, policy, route.params.policyID],

diff --git a/tests/e2e/utils/installApp.ts b/tests/e2e/utils/installApp.ts
@@ -17,7 +17,7 @@ export default function (packageName: string, path: string, platform = 'android'
         execAsync(`adb uninstall ${packageName}`)
             .catch((error: ExecException) => {
                 // Ignore errors
-                Logger.warn('Failed to uninstall app:', error);
+                Logger.warn('Failed to uninstall app:', error.message);
             })
             // eslint-disable-next-line @typescript-eslint/no-misused-promises
             .finally(() => execAsync(`adb install ${path}`))