Page MenuHomePhabricator
Differential D13161

[lib] implement retrying DM text messages
ClosedPublic
Actions

Authored by kamil on Aug 23 2024, 7:18 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 9, 7:12 PM
Unknown Object (File)
Sat, Nov 9, 3:59 PM
Unknown Object (File)
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)
Oct 25 2024, 2:37 AM
Unknown Object (File)
Sep 25 2024, 3:09 PM
View All 48 Files
Subscribers
ashoat

Details

Reviewers
tomek
inka
Commits
rCOMMa242a4493a2f: [lib] implement retrying DM text messages
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil created this revision.Aug 23 2024, 7:18 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptAug 23 2024, 7:18 AM
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 ↗(On Diff #43618)
Harbormaster completed remote builds in B31245: Diff 43618.Aug 23 2024, 7:35 AM
tomek accepted this revision.Aug 26 2024, 4:42 AM
tomek added inline comments.
native/chat/failed-send.react.js
155 ↗(On Diff #43618)

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
kamil added inline comments.Aug 26 2024, 7:47 AM
native/chat/failed-send.react.js
155 ↗(On Diff #43618)

I think this is implemented here using retryingText

kamil updated this revision to Diff 43644.Aug 26 2024, 8:22 AM

update

Harbormaster failed remote builds in B31269: Diff 43644!Aug 26 2024, 8:49 AM
This revision was landed with ongoing or failed builds.Aug 26 2024, 9:05 AM
Closed by commit rCOMMa242a4493a2f: [lib] implement retrying DM text messages (authored by kamil). · Explain Why
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMMa242a4493a2f: [lib] implement retrying DM text messages.
Harbormaster completed remote builds in B31269: Diff 43644.Aug 26 2024, 9:06 AM

Revision Contents
Changeset List

PathSize
lib/
shared/
dm-ops/
35 lines
native/
chat/
28 lines
web/
chat/
21 lines

Diff 43661

View Options

lib/shared/dm-ops/process-dm-ops.js

Loading...
View Options

native/chat/failed-send.react.js

Loading...
View Options

web/chat/failed-send.react.js

Loading...