Skip to content

Commit

Permalink
Merge pull request #45049 from Expensify/beaman-unapproveButtonFixes
Browse files Browse the repository at this point in the history
[CP Staging] Show Unapprove button only for expense reports & not in transaction thread header if > 1 transaction
  • Loading branch information
Beamanator authored Jul 9, 2024
2 parents 7c37886 + 9fafd6b commit e4e7baa
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/pages/ReportDetailsPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -200,9 +200,9 @@ function ReportDetailsPage({policies, report, session, personalDetails}: ReportD
const shouldShowDeleteButton = shouldShowTaskDeleteButton || canDeleteRequest;

const canUnapproveRequest =
ReportUtils.isMoneyRequestReport(moneyRequestReport) &&
(ReportUtils.isReportManager(moneyRequestReport) || isPolicyAdmin) &&
(ReportUtils.isReportApproved(moneyRequestReport) || ReportUtils.isReportManuallyReimbursed(moneyRequestReport));
ReportUtils.isExpenseReport(report) &&
(ReportUtils.isReportManager(report) || isPolicyAdmin) &&
(ReportUtils.isReportApproved(report) || ReportUtils.isReportManuallyReimbursed(report));

useEffect(() => {
if (canDeleteRequest) {
Expand Down

0 comments on commit e4e7baa

Please sign in to comment.