[lib] implement reply count for composable DM messages
ClosedPublic
Actions

Authored by kamil on Sep 23 2024, 12:52 PM.

Details

Reviewers

tomek
marcin

Commits

rCOMM887af9313d3c: [lib] implement reply count for composable DM messages

Summary

ENG-9273.

Depends on D13426

Test Plan

Reply count on sender.
Reply count on receiver.
Reply count when retrying.

Tested both web and native.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Sep 23 2024, 12:52 PM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptSep 23 2024, 12:52 PM

Harbormaster completed remote builds in B31775: Diff 44422.Sep 23 2024, 1:17 PM

kamil published this revision for review.Sep 23 2024, 1:21 PM

tomek accepted this revision.Sep 23 2024, 1:38 PM

tomek added inline comments.

lib/shared/dm-ops/process-dm-ops.js
323–329 ↗	(On Diff #44422)	This probably can be simplified

This revision is now accepted and ready to land.Sep 23 2024, 1:38 PM

kamil added inline comments.Sep 23 2024, 3:33 PM

lib/shared/dm-ops/process-dm-ops.js
323–329 ↗	(On Diff #44422)	There is the most readable way when Flow is not complaining

Closed by commit rCOMM887af9313d3c: [lib] implement reply count for composable DM messages. · Explain WhySep 23 2024, 3:49 PM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM887af9313d3c: [lib] implement reply count for composable DM messages.

Revision Contents
Changeset List

Path

Size

lib/

shared/

dm-ops/

process-dm-ops.js

26 lines

Diff 44441

View Options

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

[lib] implement reply count for composable DM messagesClosedPublicActions