Merge pull request Expensify#63326 from nabi-ebrahimi/fix/approval-flow-missing-approve-button-approver-c

rlinoz · web-flow · commit 4a939b2b1557 · 2025-06-04T09:32:47.000-03:00
fix:  Missing Approve button for Aprrover C in the report detail or preview
diff --git a/src/libs/ReportPrimaryActionUtils.ts b/src/libs/ReportPrimaryActionUtils.ts
@@ -94,7 +94,7 @@ function isApproveAction(report: Report, reportTransactions: Transaction[], poli
         return false;
     }
     const isExpenseReport = isExpenseReportUtils(report);
-    const isReportApprover = isApproverUtils(policy, currentUserAccountID);
+    const isReportApprover = isApproverUtils(policy, currentUserAccountID) || managerID === currentUserAccountID;
     const isApprovalEnabled = policy?.approvalMode && policy.approvalMode !== CONST.POLICY.APPROVAL_MODE.OPTIONAL;
 
     if (!isExpenseReport || !isReportApprover || !isApprovalEnabled || reportTransactions.length === 0) {

Original file line number	Diff line number	Diff line change
`@@ -94,7 +94,7 @@ function isApproveAction(report: Report, reportTransactions: Transaction[], poli`
`94`	`94`	`return false;`
`95`	`95`	`}`
`96`	`96`	`const isExpenseReport = isExpenseReportUtils(report);`
`97`		`- const isReportApprover = isApproverUtils(policy, currentUserAccountID);`
	`97`	`+ const isReportApprover = isApproverUtils(policy, currentUserAccountID) \|\| managerID === currentUserAccountID;`
`98`	`98`	`const isApprovalEnabled = policy?.approvalMode && policy.approvalMode !== CONST.POLICY.APPROVAL_MODE.OPTIONAL;`
`99`	`99`
`100`	`100`	`if (!isExpenseReport \|\| !isReportApprover \|\| !isApprovalEnabled \|\| reportTransactions.length === 0) {`