Merge pull request #3734 from Expensify/clem-fix-sequence-number

Send the sequenceNumber when deleting a report comment
Expensify · Jun 23, 2021 · 8f8d835 · 8f8d835
2 parents ad41508 + afd5447
commit 8f8d835
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/libs/actions/Report.js b/src/libs/actions/Report.js
@@ -1073,9 +1073,10 @@ function addAction(reportID, text, file) {
  */
 function deleteReportComment(reportID, reportAction) {
     // Optimistic Response
+    const sequenceNumber = reportAction.sequenceNumber;
     const reportActionsToMerge = {};
     const oldMessage = {...reportAction.message};
-    reportActionsToMerge[reportAction.sequenceNumber] = {
+    reportActionsToMerge[sequenceNumber] = {
         ...reportAction,
         message: [
             {
@@ -1093,11 +1094,12 @@ function deleteReportComment(reportID, reportAction) {
         reportID,
         reportActionID: reportAction.reportActionID,
         reportComment: '',
+        sequenceNumber,
     })
         .then((response) => {
             if (response.jsonCode !== 200) {
                 // Reverse Optimistic Response
-                reportActionsToMerge[reportAction.sequenceNumber] = {
+                reportActionsToMerge[sequenceNumber] = {
                     ...reportAction,
                     message: oldMessage,
                 };