Skip to content

Commit 33bf01d

Browse files
BeamanatorOSBotify
authored andcommitted
Merge pull request #45049 from Expensify/beaman-unapproveButtonFixes
[CP Staging] Show Unapprove button only for expense reports & not in transaction thread header if > 1 transaction (cherry picked from commit e4e7baa)
1 parent ae26e23 commit 33bf01d

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/pages/ReportDetailsPage.tsx

+3-3
Original file line numberDiff line numberDiff line change
@@ -200,9 +200,9 @@ function ReportDetailsPage({policies, report, session, personalDetails}: ReportD
200200
const shouldShowDeleteButton = shouldShowTaskDeleteButton || canDeleteRequest;
201201

202202
const canUnapproveRequest =
203-
ReportUtils.isMoneyRequestReport(moneyRequestReport) &&
204-
(ReportUtils.isReportManager(moneyRequestReport) || isPolicyAdmin) &&
205-
(ReportUtils.isReportApproved(moneyRequestReport) || ReportUtils.isReportManuallyReimbursed(moneyRequestReport));
203+
ReportUtils.isExpenseReport(report) &&
204+
(ReportUtils.isReportManager(report) || isPolicyAdmin) &&
205+
(ReportUtils.isReportApproved(report) || ReportUtils.isReportManuallyReimbursed(report));
206206

207207
useEffect(() => {
208208
if (canDeleteRequest) {

0 commit comments

Comments
 (0)