Paths

Table of Contentst

Differential D12806

[lib] DMOperationSpec for reaction operation
ClosedPublic
Actions

Authored by tomek on Jul 19 2024, 12:52 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Nov 11, 1:42 PM

	Unknown Object (File)
	Mon, Nov 11, 11:27 AM

	Unknown Object (File)
	Mon, Nov 11, 9:28 AM

	Unknown Object (File)
	Mon, Nov 11, 7:44 AM

	Unknown Object (File)
	Mon, Nov 11, 6:39 AM

	Unknown Object (File)
	Mon, Nov 11, 4:56 AM

	Unknown Object (File)
	Sun, Nov 10, 10:07 PM

	Unknown Object (File)
	Fri, Nov 8, 6:00 AM

View All 84 Files

Subscribers

Details

Reviewers

kamil
inka

Commits

rCOMM20323899bb24: [lib] DMOperationSpec for reaction operation

Summary

Create the spec.

https://linear.app/comm/issue/ENG-8732/create-dmoperations-for-reactions

Test Plan

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

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.Jul 19 2024, 12:52 PM

Herald added a subscriber: ashoat. · View Herald TranscriptJul 19 2024, 12:52 PM

Rename

Harbormaster completed remote builds in B30516: Diff 42551.Jul 19 2024, 1:09 PM

Harbormaster completed remote builds in B30518: Diff 42553.Jul 19 2024, 1:19 PM

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

Rename file

Harbormaster completed remote builds in B30520: Diff 42555.Jul 19 2024, 1:49 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

In D12806#363499, @ashoat wrote:

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

Add to specs

Harbormaster completed remote builds in B30612: Diff 42699.Jul 23 2024, 7:44 AM

inka accepted this revision.Jul 25 2024, 1:03 AM

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

Rename

Harbormaster completed remote builds in B30700: Diff 42817.Jul 26 2024, 4:42 AM

Closed by commit rCOMM20323899bb24: [lib] DMOperationSpec for reaction operation (authored by tomek). · Explain WhyJul 26 2024, 5:22 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM20323899bb24: [lib] DMOperationSpec for reaction operation.

Revision Contents
Changeset List

Path

Size

lib/

shared/

dm-ops/

2 lines

send-reaction-message-spec.js

54 lines

types/

27 lines

Diff 42825

lib/shared/dm-ops/dm-op-specs.js

Loading...

lib/shared/dm-ops/send-reaction-message-spec.js

Loading...

lib/types/dm-ops.js

Loading...