Fix bugs surrounding report name when deleting first thread comment

ojasjadhav2 · Jul 24, 2023 · 93e5812 · 93e5812
1 parent c3fd1ea
commit 93e5812
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/src/libs/actions/Report.js b/src/libs/actions/Report.js
@@ -897,7 +897,7 @@ function deleteReportComment(reportID, reportAction) {
         lastMessageText: '',
         lastVisibleActionCreated: '',
     };
-    if (reportAction.reportActionID && reportAction.childVisibleActionCount > 0) {
+    if (reportAction.reportActionID && reportAction.childVisibleActionCount === 0) {
         optimisticReport = {
             lastMessageTranslationKey: '',
             lastMessageText: '',
@@ -963,6 +963,16 @@ function deleteReportComment(reportID, reportAction) {
         optimisticData.push(optimisticParentReportData);
     }
 
+    // If the reportID doesn't equal the originalReportID then this is the first comment on a thread report and we need to trigger
+    // some sort of update to let the LHN know that the parentReportAction is now deleted.
+    if (reportID !== originalReportID) {
+        optimisticData.push({
+            onyxMethod: Onyx.METHOD.MERGE,
+            key: `${ONYXKEYS.COLLECTION.REPORT}${reportID}`,
+            value: {isParentReportActionDeleted: true},
+        });
+    }
+
     const parameters = {
         reportID: originalReportID,
         reportActionID,