Skip to content

Commit

Permalink
Merge pull request Expensify#28880 from Expensify/vit-accessNotificat…
Browse files Browse the repository at this point in the history
…ionSafely

[NoQA] Access the report action safely in notification logic
  • Loading branch information
mountiny authored Oct 8, 2023
2 parents 389d7b0 + cf9a5d5 commit e0ab038
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/libs/actions/Report.js
Original file line number Diff line number Diff line change
Expand Up @@ -1661,7 +1661,7 @@ function shouldShowReportActionNotification(reportID, action = null, isRemote =

// Only show notifications for supported types of report actions
if (!ReportActionsUtils.isNotifiableReportAction(action)) {
Log.info(`${tag} No notification because this action type is not supported`, false, {actionName: action.actionName});
Log.info(`${tag} No notification because this action type is not supported`, false, {actionName: lodashGet(action, 'actionName')});
return false;
}

Expand Down

0 comments on commit e0ab038

Please sign in to comment.