Merge pull request #22848 from Expensify/yuwen-delegateDetails

Make sure we have delegate details before displaying them in the case of a delegate action
Expensify · Jul 17, 2023 · 653ef72 · 653ef72
2 parents f638162 + c761435
commit 653ef72
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/pages/home/report/ReportActionItemSingle.js b/src/pages/home/report/ReportActionItemSingle.js
@@ -82,9 +82,9 @@ function ReportActionItemSingle(props) {
         displayName = ReportUtils.getPolicyName(props.report);
         actorHint = displayName;
         avatarSource = ReportUtils.getWorkspaceAvatar(props.report);
-    } else if (props.action.delegateAccountID) {
-        // We replace the actor's email, name, and avatar with the Copilot manually for now. This will be improved upon when
-        // the Copilot feature is implemented.
+    } else if (props.action.delegateAccountID && props.personalDetailsList[props.action.delegateAccountID]) {
+        // We replace the actor's email, name, and avatar with the Copilot manually for now. And only if we have their
+        // details. This will be improved upon when the Copilot feature is implemented.
         const delegateDetails = props.personalDetailsList[props.action.delegateAccountID];
         const delegateDisplayName = delegateDetails.displayName;
         actorHint = `${delegateDisplayName} (${props.translate('reportAction.asCopilot')} ${displayName})`;