remove the manager check

Expensify · Jul 17, 2024 · f3b3437 · f3b3437
1 parent 68ab5c6
commit f3b3437
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 24 deletions.
diff --git a/src/components/MoneyReportHeader.tsx b/src/components/MoneyReportHeader.tsx
@@ -55,22 +55,11 @@ type MoneyReportHeaderProps = {
     /** Whether we should display the header as in narrow layout */
     shouldUseNarrowLayout?: boolean;
 
-    /** The accountID of the current user */
-    currentUserAccountID: number;
-
     /** Method to trigger when pressing close button of the header */
     onBackButtonPress: () => void;
 };
 
-function MoneyReportHeader({
-    policy,
-    report: moneyRequestReport,
-    transactionThreadReportID,
-    reportActions,
-    shouldUseNarrowLayout = false,
-    onBackButtonPress,
-    currentUserAccountID,
-}: MoneyReportHeaderProps) {
+function MoneyReportHeader({policy, report: moneyRequestReport, transactionThreadReportID, reportActions, shouldUseNarrowLayout = false, onBackButtonPress}: MoneyReportHeaderProps) {
     const [chatReport] = useOnyx(`${ONYXKEYS.COLLECTION.REPORT}${moneyRequestReport.chatReportID}`);
     const [nextStep] = useOnyx(`${ONYXKEYS.COLLECTION.NEXT_STEP}${moneyRequestReport.reportID}`);
     const [transactionThreadReport] = useOnyx(`${ONYXKEYS.COLLECTION.REPORT}${transactionThreadReportID}`);
@@ -129,9 +118,8 @@ function MoneyReportHeader({
 
     const shouldShowSubmitButton = isDraft && reimbursableSpend !== 0 && !allHavePendingRTERViolation;
 
-    const isManager = currentUserAccountID === moneyRequestReport?.managerID;
     const isAdmin = policy?.role === CONST.POLICY.ROLE.ADMIN;
-    const shouldShowExportIntegrationButton = !shouldShowPayButton && !shouldShowSubmitButton && connectedIntegration && (isManager || isAdmin);
+    const shouldShowExportIntegrationButton = !shouldShowPayButton && !shouldShowSubmitButton && connectedIntegration && isAdmin;
 
     const shouldShowSettlementButton = (shouldShowPayButton || shouldShowApproveButton) && !allHavePendingRTERViolation && !shouldShowExportIntegrationButton;
 

diff --git a/src/components/ReportActionItem/ReportPreview.tsx b/src/components/ReportActionItem/ReportPreview.tsx
@@ -47,8 +47,6 @@ type ReportPreviewOnyxProps = {
     /** The policy tied to the expense report */
     policy: OnyxEntry<Policy>;
 
-    session: OnyxEntry<Session>;
-
     /** ChatReport associated with iouReport */
     chatReport: OnyxEntry<Report>;
 
@@ -98,7 +96,6 @@ function ReportPreview({
     iouReport,
     policy,
     iouReportID,
-    session,
     policyID,
     chatReportID,
     chatReport,
@@ -118,7 +115,6 @@ function ReportPreview({
     const {canUseViolations} = usePermissions();
     const {isOffline} = useNetwork();
 
-    const currentUserAccountID = session?.accountID;
     const {hasMissingSmartscanFields, areAllRequestsBeingSmartScanned, hasOnlyTransactionsWithPendingRoutes, hasNonReimbursableTransactions} = useMemo(
         () => ({
             hasMissingSmartscanFields: ReportUtils.hasMissingSmartscanFields(iouReportID),
@@ -345,9 +341,8 @@ function ReportPreview({
      */
     const connectedIntegration = PolicyUtils.getConnectedIntegration(policy);
 
-    const isManager = currentUserAccountID === iouReport?.managerID;
     const isAdmin = policy?.role === CONST.POLICY.ROLE.ADMIN;
-    const shouldShowExportIntegrationButton = !shouldShowPayButton && !shouldShowSubmitButton && connectedIntegration && (isManager || isAdmin);
+    const shouldShowExportIntegrationButton = !shouldShowPayButton && !shouldShowSubmitButton && connectedIntegration && isAdmin;
 
     return (
         <OfflineWithFeedback
@@ -517,7 +512,4 @@ export default withOnyx<ReportPreviewProps, ReportPreviewOnyxProps>({
     userWallet: {
         key: ONYXKEYS.USER_WALLET,
     },
-    session: {
-        key: ONYXKEYS.SESSION,
-    },
 })(ReportPreview);
diff --git a/src/pages/home/ReportScreen.tsx b/src/pages/home/ReportScreen.tsx
@@ -358,7 +358,6 @@ function ReportScreen({route, currentReportID = '', navigation}: ReportScreenPro
                 reportActions={reportActions}
                 shouldUseNarrowLayout={shouldUseNarrowLayout}
                 onBackButtonPress={onBackButtonPress}
-                currentUserAccountID={currentUserAccountID}
             />
         );
     }