Skip to content

Commit 8f8d835

Browse files
authored
Merge pull request #3734 from Expensify/clem-fix-sequence-number
Send the sequenceNumber when deleting a report comment
2 parents ad41508 + afd5447 commit 8f8d835

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/libs/actions/Report.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1073,9 +1073,10 @@ function addAction(reportID, text, file) {
10731073
*/
10741074
function deleteReportComment(reportID, reportAction) {
10751075
// Optimistic Response
1076+
const sequenceNumber = reportAction.sequenceNumber;
10761077
const reportActionsToMerge = {};
10771078
const oldMessage = {...reportAction.message};
1078-
reportActionsToMerge[reportAction.sequenceNumber] = {
1079+
reportActionsToMerge[sequenceNumber] = {
10791080
...reportAction,
10801081
message: [
10811082
{
@@ -1093,11 +1094,12 @@ function deleteReportComment(reportID, reportAction) {
10931094
reportID,
10941095
reportActionID: reportAction.reportActionID,
10951096
reportComment: '',
1097+
sequenceNumber,
10961098
})
10971099
.then((response) => {
10981100
if (response.jsonCode !== 200) {
10991101
// Reverse Optimistic Response
1100-
reportActionsToMerge[reportAction.sequenceNumber] = {
1102+
reportActionsToMerge[sequenceNumber] = {
11011103
...reportAction,
11021104
message: oldMessage,
11031105
};

0 commit comments

Comments
 (0)