Skip to content

Commit 3dfb0b9

Browse files
authored
Merge pull request Expensify#69618 from Expensify/scott-removeVacationDelegateBeta
Remove beta & check for vacation delegate
2 parents 48c4eb1 + 26f519d commit 3dfb0b9

2 files changed

Lines changed: 33 additions & 40 deletions

File tree

src/CONST/index.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -696,7 +696,6 @@ const CONST = {
696696
EUR_BILLING: 'eurBilling',
697697
MANUAL_DISTANCE: 'manualDistance',
698698
NO_OPTIMISTIC_TRANSACTION_THREADS: 'noOptimisticTransactionThreads',
699-
VACATION_DELEGATE: 'vacationDelegate',
700699
UBER_FOR_BUSINESS: 'uberForBusiness',
701700
DUPLICATE_WORKSPACE: 'duplicatePolicyNewDot',
702701
},

src/pages/settings/Profile/CustomStatus/StatusPage.tsx

Lines changed: 33 additions & 39 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ import useAutoFocusInput from '@hooks/useAutoFocusInput';
1818
import useCurrentUserPersonalDetails from '@hooks/useCurrentUserPersonalDetails';
1919
import useLocalize from '@hooks/useLocalize';
2020
import useOnyx from '@hooks/useOnyx';
21-
import usePermissions from '@hooks/usePermissions';
2221
import useStyleUtils from '@hooks/useStyleUtils';
2322
import useTheme from '@hooks/useTheme';
2423
import useThemeStyles from '@hooks/useThemeStyles';
@@ -52,9 +51,6 @@ function StatusPage() {
5251
const vacationDelegatePersonalDetails = getPersonalDetailByEmail(vacationDelegate?.delegate ?? '');
5352
const formattedDelegateLogin = formatPhoneNumber(vacationDelegatePersonalDetails?.login ?? '');
5453

55-
const {isBetaEnabled} = usePermissions();
56-
const isVacationDelegateEnabled = isBetaEnabled(CONST.BETAS.VACATION_DELEGATE);
57-
5854
const currentUserEmojiCode = currentUserPersonalDetails?.status?.emojiCode ?? '';
5955
const currentUserStatusText = currentUserPersonalDetails?.status?.text ?? '';
6056
const currentUserClearAfter = currentUserPersonalDetails?.status?.clearAfter ?? '';
@@ -246,41 +242,39 @@ function StatusPage() {
246242
/>
247243
)}
248244
</View>
249-
{isVacationDelegateEnabled && (
250-
<View style={[styles.mb2, styles.mt6]}>
251-
<Text style={[styles.mh5]}>{translate('statusPage.setVacationDelegate')}</Text>
252-
{hasVacationDelegate && <Text style={[styles.mh5, styles.mt6, styles.mutedTextLabel]}>{translate('statusPage.vacationDelegate')}</Text>}
253-
{hasVacationDelegate ? (
254-
<OfflineWithFeedback
255-
pendingAction={vacationDelegate?.pendingAction}
256-
errors={vacationDelegate?.errors}
257-
errorRowStyles={styles.mh5}
258-
onClose={() => clearVacationDelegateError(vacationDelegate?.previousDelegate)}
259-
>
260-
<MenuItem
261-
title={vacationDelegatePersonalDetails?.displayName ?? fallbackVacationDelegateLogin}
262-
description={fallbackVacationDelegateLogin}
263-
avatarID={vacationDelegatePersonalDetails?.accountID ?? CONST.DEFAULT_NUMBER_ID}
264-
icon={vacationDelegatePersonalDetails?.avatar ?? Expensicons.FallbackAvatar}
265-
iconType={CONST.ICON_TYPE_AVATAR}
266-
numberOfLinesDescription={1}
267-
shouldShowRightIcon
268-
onPress={() => Navigation.navigate(ROUTES.SETTINGS_VACATION_DELEGATE)}
269-
containerStyle={styles.pr2}
270-
/>
271-
</OfflineWithFeedback>
272-
) : (
273-
<View style={[styles.mt1]}>
274-
<MenuItem
275-
description={translate('statusPage.vacationDelegate')}
276-
shouldShowRightIcon
277-
onPress={() => Navigation.navigate(ROUTES.SETTINGS_VACATION_DELEGATE)}
278-
containerStyle={styles.pr2}
279-
/>
280-
</View>
281-
)}
282-
</View>
283-
)}
245+
<View style={[styles.mb2, styles.mt6]}>
246+
<Text style={[styles.mh5]}>{translate('statusPage.setVacationDelegate')}</Text>
247+
{hasVacationDelegate && <Text style={[styles.mh5, styles.mt6, styles.mutedTextLabel]}>{translate('statusPage.vacationDelegate')}</Text>}
248+
{hasVacationDelegate ? (
249+
<OfflineWithFeedback
250+
pendingAction={vacationDelegate?.pendingAction}
251+
errors={vacationDelegate?.errors}
252+
errorRowStyles={styles.mh5}
253+
onClose={() => clearVacationDelegateError(vacationDelegate?.previousDelegate)}
254+
>
255+
<MenuItem
256+
title={vacationDelegatePersonalDetails?.displayName ?? fallbackVacationDelegateLogin}
257+
description={fallbackVacationDelegateLogin}
258+
avatarID={vacationDelegatePersonalDetails?.accountID ?? CONST.DEFAULT_NUMBER_ID}
259+
icon={vacationDelegatePersonalDetails?.avatar ?? Expensicons.FallbackAvatar}
260+
iconType={CONST.ICON_TYPE_AVATAR}
261+
numberOfLinesDescription={1}
262+
shouldShowRightIcon
263+
onPress={() => Navigation.navigate(ROUTES.SETTINGS_VACATION_DELEGATE)}
264+
containerStyle={styles.pr2}
265+
/>
266+
</OfflineWithFeedback>
267+
) : (
268+
<View style={[styles.mt1]}>
269+
<MenuItem
270+
description={translate('statusPage.vacationDelegate')}
271+
shouldShowRightIcon
272+
onPress={() => Navigation.navigate(ROUTES.SETTINGS_VACATION_DELEGATE)}
273+
containerStyle={styles.pr2}
274+
/>
275+
</View>
276+
)}
277+
</View>
284278
</FormProvider>
285279
</ScreenWrapper>
286280
);

0 commit comments

Comments
 (0)