Skip to content

Commit 4e5a3b9

Browse files
authored
Merge pull request Expensify#64104 from FitseTLT/fix-restrict-custom-fields-setting-only-for-control
2 parents 13866e3 + 8afe831 commit 4e5a3b9

2 files changed

Lines changed: 22 additions & 17 deletions

File tree

src/pages/workspace/members/WorkspaceMemberCustomFieldPage.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ function WorkspaceMemberCustomFieldPage({policy, route, personalDetails}: Worksp
5151
return (
5252
<AccessOrNotFoundWrapper
5353
policyID={policyID}
54-
accessVariants={[CONST.POLICY.ACCESS_VARIANTS.ADMIN, CONST.POLICY.ACCESS_VARIANTS.PAID]}
54+
accessVariants={[CONST.POLICY.ACCESS_VARIANTS.ADMIN, CONST.POLICY.ACCESS_VARIANTS.CONTROL]}
5555
>
5656
<ScreenWrapper
5757
testID="WorkspaceMemberCustomFieldPage"

src/pages/workspace/members/WorkspaceMemberDetailsPage.tsx

Lines changed: 21 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ import {getAllCardsForWorkspace, getCardFeedIcon, getCompanyFeeds, getPlaidInsti
3131
import {convertToDisplayString} from '@libs/CurrencyUtils';
3232
import type {PlatformStackScreenProps} from '@libs/Navigation/PlatformStackNavigation/types';
3333
import {getDisplayNameOrDefault, getPhoneNumber} from '@libs/PersonalDetailsUtils';
34+
import {isControlPolicy} from '@libs/PolicyUtils';
3435
import shouldRenderTransferOwnerButton from '@libs/shouldRenderTransferOwnerButton';
3536
import {convertPolicyEmployeesToApprovalWorkflows, updateWorkflowDataOnApproverRemoval} from '@libs/WorkflowUtils';
3637
import Navigation from '@navigation/Navigation';
@@ -360,22 +361,26 @@ function WorkspaceMemberDetailsPage({personalDetails, policy, route}: WorkspaceM
360361
shouldShowRightIcon
361362
onPress={openRoleSelectionModal}
362363
/>
363-
<OfflineWithFeedback pendingAction={member?.pendingFields?.employeeUserID}>
364-
<MenuItemWithTopDescription
365-
description={translate('workspace.common.customField1')}
366-
title={member?.employeeUserID}
367-
shouldShowRightIcon
368-
onPress={() => Navigation.navigate(ROUTES.WORKSPACE_CUSTOM_FIELDS.getRoute(policyID, accountID, 'customField1'))}
369-
/>
370-
</OfflineWithFeedback>
371-
<OfflineWithFeedback pendingAction={member?.pendingFields?.employeePayrollID}>
372-
<MenuItemWithTopDescription
373-
description={translate('workspace.common.customField2')}
374-
title={member?.employeePayrollID}
375-
shouldShowRightIcon
376-
onPress={() => Navigation.navigate(ROUTES.WORKSPACE_CUSTOM_FIELDS.getRoute(policyID, accountID, 'customField2'))}
377-
/>
378-
</OfflineWithFeedback>
364+
{isControlPolicy(policy) && (
365+
<>
366+
<OfflineWithFeedback pendingAction={member?.pendingFields?.employeeUserID}>
367+
<MenuItemWithTopDescription
368+
description={translate('workspace.common.customField1')}
369+
title={member?.employeeUserID}
370+
shouldShowRightIcon
371+
onPress={() => Navigation.navigate(ROUTES.WORKSPACE_CUSTOM_FIELDS.getRoute(policyID, accountID, 'customField1'))}
372+
/>
373+
</OfflineWithFeedback>
374+
<OfflineWithFeedback pendingAction={member?.pendingFields?.employeePayrollID}>
375+
<MenuItemWithTopDescription
376+
description={translate('workspace.common.customField2')}
377+
title={member?.employeePayrollID}
378+
shouldShowRightIcon
379+
onPress={() => Navigation.navigate(ROUTES.WORKSPACE_CUSTOM_FIELDS.getRoute(policyID, accountID, 'customField2'))}
380+
/>
381+
</OfflineWithFeedback>
382+
</>
383+
)}
379384
<MenuItem
380385
style={styles.mb5}
381386
title={translate('common.profile')}

0 commit comments

Comments
 (0)