From 698dbd9095901a0f87acdfc6df86e0690a14a43f Mon Sep 17 00:00:00 2001
From: Sibtain Ali <allroundexperts@gmail.com>
Date: Mon, 22 Jan 2024 19:07:54 +0500
Subject: [PATCH 1/2] feat: allow policy admins to edit the report fields of a
 non-settled report

---
 .../ReportActionItem/MoneyReportView.tsx      | 29 ++++++++++++++-----
 1 file changed, 22 insertions(+), 7 deletions(-)

diff --git a/src/components/ReportActionItem/MoneyReportView.tsx b/src/components/ReportActionItem/MoneyReportView.tsx
index 4fcca3e518a5..7e0ff9487232 100644
--- a/src/components/ReportActionItem/MoneyReportView.tsx
+++ b/src/components/ReportActionItem/MoneyReportView.tsx
@@ -1,6 +1,8 @@
 import React, {useMemo} from 'react';
 import type {StyleProp, TextStyle} from 'react-native';
 import {View} from 'react-native';
+import {withOnyx} from 'react-native-onyx';
+import type {OnyxCollection} from 'react-native-onyx';
 import Icon from '@components/Icon';
 import * as Expensicons from '@components/Icon/Expensicons';
 import MenuItemWithTopDescription from '@components/MenuItemWithTopDescription';
@@ -17,9 +19,10 @@ import * as CurrencyUtils from '@libs/CurrencyUtils';
 import * as ReportUtils from '@libs/ReportUtils';
 import AnimatedEmptyStateBackground from '@pages/home/report/AnimatedEmptyStateBackground';
 import variables from '@styles/variables';
-import type {PolicyReportField, Report} from '@src/types/onyx';
+import ONYXKEYS from '@src/ONYXKEYS';
+import type {Policy, PolicyReportField, Report} from '@src/types/onyx';
 
-type MoneyReportViewProps = {
+type MoneyReportViewComponentProps = {
     /** The report currently being looked at */
     report: Report;
 
@@ -30,7 +33,14 @@ type MoneyReportViewProps = {
     shouldShowHorizontalRule: boolean;
 };
 
-function MoneyReportView({report, policyReportFields, shouldShowHorizontalRule}: MoneyReportViewProps) {
+type MoneyReportViewOnyxProps = {
+    /** Policies that the user is part of */
+    policies: OnyxCollection<Policy>;
+};
+
+type MoneyReportViewProps = MoneyReportViewComponentProps & MoneyReportViewOnyxProps;
+
+function MoneyReportView({report, policyReportFields, shouldShowHorizontalRule, policies}: MoneyReportViewProps) {
     const theme = useTheme();
     const styles = useThemeStyles();
     const StyleUtils = useStyleUtils();
@@ -57,7 +67,7 @@ function MoneyReportView({report, policyReportFields, shouldShowHorizontalRule}:
         () => policyReportFields.sort(({orderWeight: firstOrderWeight}, {orderWeight: secondOrderWeight}) => firstOrderWeight - secondOrderWeight),
         [policyReportFields],
     );
-
+    const isAdmin = ReportUtils.isPolicyAdmin(report.policyID ?? '', policies);
     return (
         <View style={[StyleUtils.getReportWelcomeContainerStyle(isSmallScreenWidth, true)]}>
             <AnimatedEmptyStateBackground />
@@ -65,6 +75,7 @@ function MoneyReportView({report, policyReportFields, shouldShowHorizontalRule}:
                 {canUseReportFields &&
                     sortedPolicyReportFields.map((reportField) => {
                         const title = ReportUtils.getReportFieldTitle(report, reportField);
+                        const isDisabled = !isAdmin || isSettled;
                         return (
                             <OfflineWithFeedback
                                 pendingAction={report.pendingFields?.[reportField.fieldID]}
@@ -74,8 +85,8 @@ function MoneyReportView({report, policyReportFields, shouldShowHorizontalRule}:
                                     description={reportField.name}
                                     title={title}
                                     onPress={() => {}}
-                                    shouldShowRightIcon
-                                    disabled={false}
+                                    shouldShowRightIcon={!isDisabled}
+                                    disabled={isDisabled}
                                     wrapperStyle={[styles.pv2, styles.taskDescriptionMenuItem]}
                                     shouldGreyOutWhenDisabled={false}
                                     numberOfLinesTitle={0}
@@ -165,4 +176,8 @@ function MoneyReportView({report, policyReportFields, shouldShowHorizontalRule}:
 
 MoneyReportView.displayName = 'MoneyReportView';
 
-export default MoneyReportView;
+export default withOnyx<MoneyReportViewProps, MoneyReportViewOnyxProps>({
+    policies: {
+        key: ONYXKEYS.COLLECTION.POLICY,
+    },
+})(MoneyReportView);

From da41efb6a64e444c7611ad528c9d702e42012dbf Mon Sep 17 00:00:00 2001
From: Sibtain Ali <allroundexperts@gmail.com>
Date: Tue, 23 Jan 2024 23:47:20 +0500
Subject: [PATCH 2/2] feat: merge with main

---
 src/components/ReportActionItem/MoneyReportView.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/ReportActionItem/MoneyReportView.tsx b/src/components/ReportActionItem/MoneyReportView.tsx
index ae3cc4c91b86..ed7c05b828a9 100644
--- a/src/components/ReportActionItem/MoneyReportView.tsx
+++ b/src/components/ReportActionItem/MoneyReportView.tsx
@@ -21,8 +21,8 @@ import * as ReportUtils from '@libs/ReportUtils';
 import AnimatedEmptyStateBackground from '@pages/home/report/AnimatedEmptyStateBackground';
 import variables from '@styles/variables';
 import ONYXKEYS from '@src/ONYXKEYS';
-import type {Policy, PolicyReportField, Report} from '@src/types/onyx';
 import ROUTES from '@src/ROUTES';
+import type {Policy, PolicyReportField, Report} from '@src/types/onyx';
 
 type MoneyReportViewComponentProps = {
     /** The report currently being looked at */