Skip to content

Commit

Permalink
Merge pull request #30317 from s-alves10/fix/issue-26012
Browse files Browse the repository at this point in the history
  • Loading branch information
thienlnam authored Oct 26, 2023
2 parents 294afaf + 05ea988 commit a5e1e3f
Showing 1 changed file with 11 additions and 8 deletions.
19 changes: 11 additions & 8 deletions src/libs/actions/Report.js
Original file line number Diff line number Diff line change
Expand Up @@ -1149,14 +1149,17 @@ function deleteReportComment(reportID, reportAction) {
},
];

// Update optimistic data for parent report action if the report is a child report
const optimisticParentReportData = ReportUtils.getOptimisticDataForParentReportAction(
originalReportID,
optimisticReport.lastVisibleActionCreated,
CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE,
);
if (!_.isEmpty(optimisticParentReportData)) {
optimisticData.push(optimisticParentReportData);
// Update optimistic data for parent report action if the report is a child report and the reportAction has no visible child
const childVisibleActionCount = reportAction.childVisibleActionCount || 0;
if (childVisibleActionCount === 0) {
const optimisticParentReportData = ReportUtils.getOptimisticDataForParentReportAction(
originalReportID,
optimisticReport.lastVisibleActionCreated,
CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE,
);
if (!_.isEmpty(optimisticParentReportData)) {
optimisticData.push(optimisticParentReportData);
}
}

// Check to see if the report action we are deleting is the first comment on a thread report. In this case, we need to trigger
Expand Down

0 comments on commit a5e1e3f

Please sign in to comment.