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

Authored by kamil on Aug 27 2024, 7: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.Aug 27 2024, 7:23 PM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptAug 27 2024, 7:23 PM

Harbormaster completed remote builds in B31310: Diff 43717.Aug 27 2024, 7:41 PM

kamil published this revision for review.Aug 28 2024, 12:35 PM

tomek accepted this revision.Aug 28 2024, 12:41 PM

This revision is now accepted and ready to land.Aug 28 2024, 12:41 PM

Closed by commit rCOMM062159e457ce: [lib] add option to mark `DMOperationSpecification` as `sendOnly`. · Explain WhyAug 28 2024, 1:28 PM

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.Aug 28 2024, 3:37 PM

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