Page MenuHomePhabricator

[lib] implement reply count for composable DM messages
ClosedPublic

Authored by kamil on Sep 23 2024, 5:52 AM.
Tags
None
Referenced Files
F3299807: D13427.id44422.diff
Sun, Nov 17, 3:42 PM
Unknown Object (File)
Fri, Nov 8, 7:03 PM
Unknown Object (File)
Fri, Nov 8, 7:03 PM
Unknown Object (File)
Fri, Nov 8, 7:01 PM
Unknown Object (File)
Fri, Nov 8, 2:11 PM
Unknown Object (File)
Thu, Nov 7, 1:36 PM
Unknown Object (File)
Thu, Nov 7, 11:59 AM
Unknown Object (File)
Thu, Nov 7, 9:06 AM
Subscribers

Details

Summary

ENG-9273.

Depends on D13426

Test Plan
  1. Reply count on sender.
  2. Reply count on receiver.
  3. 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 held this revision as a draft.
kamil published this revision for review.Sep 23 2024, 6:21 AM
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, 6:38 AM
lib/shared/dm-ops/process-dm-ops.js
323–329 ↗(On Diff #44422)

There is the most readable way when Flow is not complaining