Page MenuHomePhabricator

[lib] implement retrying DM text messages
ClosedPublic

Authored by kamil on Aug 23 2024, 7:18 AM.
Tags
None
Referenced Files
F3203085: D13161.id43644.diff
Sat, Nov 9, 7:12 PM
F3201295: D13161.diff
Sat, Nov 9, 3:59 PM
F3193609: D13161.id43644.diff
Fri, Nov 8, 11:06 PM
Unknown Object (File)
Fri, Nov 8, 12:22 PM
Unknown Object (File)
Fri, Nov 8, 2:09 AM
Unknown Object (File)
Fri, Nov 8, 1:42 AM
Unknown Object (File)
Fri, Oct 25, 2:37 AM
Unknown Object (File)
Sep 25 2024, 3:09 PM
Subscribers

Details

Summary

ENG-8423.

Am alternative is to implement this in InputStateContainer, but will require a lot more changes and I don't think it's better.

Depends on D13160

Test Plan

Test retrying message

Note: This stack is tested end-to-end according to the process described here.

Diff Detail

Repository
rCOMM Comm
Branch
dms-2
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.Aug 23 2024, 7:28 AM
kamil added inline comments.
lib/shared/dm-ops/process-dm-ops.js
274
tomek added inline comments.
native/chat/failed-send.react.js
155

Should we add some protection about multiple retries of the same messages being run simultaneously?

This revision is now accepted and ready to land.Aug 26 2024, 4:42 AM
native/chat/failed-send.react.js
155

I think this is implemented here using retryingText

This revision was landed with ongoing or failed builds.Aug 26 2024, 9:05 AM
This revision was automatically updated to reflect the committed changes.