Merge pull request #28213 from azamuddin/fix/proptypes-warning-policy…

…-required-money-request-header fix: mark propTypes policy as optional and provides default value
Expensify · Oct 6, 2023 · 3e4731d · 3e4731d
2 parents eb8783c + 9f73b67
commit 3e4731d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/components/MoneyReportHeader.js b/src/components/MoneyReportHeader.js
@@ -35,7 +35,7 @@ const propTypes = {
 
         /** The role of the current user in the policy */
         role: PropTypes.string,
-    }).isRequired,
+    }),
 
     /** The chat report this report is linked to */
     chatReport: reportPropTypes,
@@ -57,6 +57,7 @@ const defaultProps = {
     session: {
         email: null,
     },
+    policy: {},
 };
 
 function MoneyReportHeader({session, personalDetails, policy, chatReport, report: moneyRequestReport, isSmallScreenWidth}) {

diff --git a/src/components/MoneyRequestHeader.js b/src/components/MoneyRequestHeader.js
@@ -30,7 +30,7 @@ const propTypes = {
     policy: PropTypes.shape({
         /** Name of the policy */
         name: PropTypes.string,
-    }).isRequired,
+    }),
 
     /** Personal details so we can get the ones for the report participants */
     personalDetails: PropTypes.objectOf(participantPropTypes).isRequired,
@@ -59,6 +59,7 @@ const defaultProps = {
     parentReport: {},
     parentReportAction: {},
     transaction: {},
+    policy: {},
 };
 
 function MoneyRequestHeader({session, parentReport, report, parentReportAction, transaction, policy, personalDetails}) {