Paths

Table of Contentst

[lib] refactor code to update reply count and unread status to function
ClosedPublic
Actions

Authored by kamil on Mon, Sep 23, 5:35 AM.

Details

Reviewers

tomek
marcin

Commits

rCOMM0a5c4d4eb6be: [lib] refactor code to update reply count and unread status to function

Summary

ENG-9273.

I need to reuse this function to bump reply count as part of useSendComposableDMOperation.

Did some additional cleanup to avoid selecting the same data multiple times.

Test Plan

Flow, functionality tested later in the stack

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Mon, Sep 23, 5:35 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptMon, Sep 23, 5:35 AM

kamil added a child revision: D13427: [lib] implement reply count for composable DM messages.Mon, Sep 23, 5:52 AM

Harbormaster completed remote builds in B31774: Diff 44421.Mon, Sep 23, 5:56 AM

kamil published this revision for review.Mon, Sep 23, 6:19 AM

tomek accepted this revision.Mon, Sep 23, 6:33 AM

tomek added inline comments.

lib/shared/dm-ops/process-dm-ops.js
224 ↗	(On Diff #44421)	I guess we don't have to check this condition

This revision is now accepted and ready to land.Mon, Sep 23, 6:33 AM

rebase

Harbormaster completed remote builds in B31779: Diff 44439.Mon, Sep 23, 8:47 AM

Closed by commit rCOMM0a5c4d4eb6be: [lib] refactor code to update reply count and unread status to function (authored by kamil). · Explain WhyMon, Sep 23, 8:49 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM0a5c4d4eb6be: [lib] refactor code to update reply count and unread status to function.

Revision Contents
Changeset List

Path

Size

lib/

shared/

dm-ops/

dm-op-utils.js

105 lines

process-dm-ops.js

96 lines

Diff 44440

View Options

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