Skip to content

Commit c15b065

Browse files
authored
Merge pull request Expensify#65488 from nkdengineer/fix/65133
fix: Error is shown after holding expense
2 parents fdf8f5e + 07be006 commit c15b065

2 files changed

Lines changed: 13 additions & 2 deletions

File tree

src/components/MoneyReportHeader.tsx

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -785,6 +785,11 @@ function MoneyReportHeader({
785785
throw new Error('Parent action does not exist');
786786
}
787787

788+
if (isDelegateAccessRestricted) {
789+
showDelegateNoAccessModal();
790+
return;
791+
}
792+
788793
changeMoneyRequestHoldStatus(requestParentReportAction);
789794
},
790795
},

src/components/MoneyRequestHeader.tsx

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import {useRoute} from '@react-navigation/native';
22
import type {ReactNode} from 'react';
3-
import React, {useCallback, useEffect, useMemo, useState} from 'react';
3+
import React, {useCallback, useContext, useEffect, useMemo, useState} from 'react';
44
import {View} from 'react-native';
55
import type {OnyxEntry} from 'react-native-onyx';
66
import type {ValueOf} from 'type-fest';
@@ -41,6 +41,7 @@ import ButtonWithDropdownMenu from './ButtonWithDropdownMenu';
4141
import type {DropdownOption} from './ButtonWithDropdownMenu/types';
4242
import ConfirmModal from './ConfirmModal';
4343
import DecisionModal from './DecisionModal';
44+
import {DelegateNoAccessContext} from './DelegateNoAccessModalProvider';
4445
import HeaderWithBackButton from './HeaderWithBackButton';
4546
import Icon from './Icon';
4647
import * as Expensicons from './Icon/Expensicons';
@@ -92,7 +93,7 @@ function MoneyRequestHeader({report, parentReportAction, policy, onBackButtonPre
9293
const isDuplicate = isDuplicateTransactionUtils(transaction);
9394
const reportID = report?.reportID;
9495
const {removeTransaction} = useSearchContext();
95-
96+
const {isDelegateAccessRestricted, showDelegateNoAccessModal} = useContext(DelegateNoAccessContext);
9697
const isReportInRHP = route.name === SCREENS.SEARCH.REPORT_RHP;
9798
const shouldDisplayTransactionNavigation = !!(reportID && isReportInRHP);
9899

@@ -214,6 +215,11 @@ function MoneyRequestHeader({report, parentReportAction, policy, onBackButtonPre
214215
throw new Error('Parent action does not exist');
215216
}
216217

218+
if (isDelegateAccessRestricted) {
219+
showDelegateNoAccessModal();
220+
return;
221+
}
222+
217223
changeMoneyRequestHoldStatus(parentReportAction);
218224
},
219225
},

0 commit comments

Comments
 (0)