Page MenuHomePhabricator

[lib] DMOperationSpec for reaction operation
ClosedPublic

Authored by tomek on Jul 19 2024, 12:52 PM.
Tags
None
Referenced Files
F3181263: D12806.id42555.diff
Fri, Nov 8, 6:00 AM
F3170824: D12806.id42818.diff
Thu, Nov 7, 1:17 PM
Unknown Object (File)
Mon, Oct 28, 9:54 AM
Unknown Object (File)
Sat, Oct 26, 8:37 AM
Unknown Object (File)
Sun, Oct 20, 8:14 AM
Unknown Object (File)
Mon, Oct 14, 11:43 AM
Unknown Object (File)
Sep 28 2024, 9:55 AM
Unknown Object (File)
Sep 28 2024, 9:28 AM
Subscribers

Details

Summary
Test Plan

Just the Flow. Further testing will be performed after it is connected with the rest of the app.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tomek requested review of this revision.Jul 19 2024, 1:19 PM

On the keyserver, when reactionMessageCreationResponder is called, it calls createMessages which in turn calls updateRepliesCount. Do we need to do something like this for DMs?

I wonder if we should have a separate task for updating the replies count. I don't think I contemplated it in sendTextMessageSpec

I wonder if we should have a separate task for updating the replies count. I don't think I contemplated it in sendTextMessageSpec

Created https://linear.app/comm/issue/ENG-8892/handle-replies-count-as-a-part-of-dmoperationspecs to track

This revision is now accepted and ready to land.Jul 25 2024, 1:03 AM