[lib] add option to mark `DMOperationSpecification` as `sendOnly`
ClosedPublic
Actions

Authored by kamil on Tue, Aug 27, 12:23 PM.

Details

Reviewers

tomek
inka

Commits

rCOMM062159e457ce: [lib] add option to mark `DMOperationSpecification` as `sendOnly`

Summary

ENG-8568

There are ops like ADD_VIEWER_TO_THREAD_MEMBERS that are only meant to be sent to other peers, and should never be processed on client which is generating this operation.

Depends on D13181

Test Plan

Tested later in the stack

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Tue, Aug 27, 12:23 PM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptTue, Aug 27, 12:23 PM

Harbormaster completed remote builds in B31310: Diff 43717.Tue, Aug 27, 12:41 PM

kamil published this revision for review.Wed, Aug 28, 5:35 AM

tomek accepted this revision.Wed, Aug 28, 5:41 AM

This revision is now accepted and ready to land.Wed, Aug 28, 5:41 AM

Closed by commit rCOMM062159e457ce: [lib] add option to mark `DMOperationSpecification` as `sendOnly` (authored by kamil). · Explain WhyWed, Aug 28, 6:28 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM062159e457ce: [lib] add option to mark `DMOperationSpecification` as `sendOnly`.

kamil added a child revision: D13197: [lib] implement adding thread members for DMs.Wed, Aug 28, 8:37 AM

Revision Contents
Changeset List

Path

Size

lib/

shared/

dm-ops/

dm-op-utils.js

1 line

process-dm-ops.js

47 lines

Diff 43741

View Options

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