[lib] Process blob holders in DM ops
Needs ReviewPublic
Actions

Authored by bartek on Fri, Sep 20, 6:30 AM.

Details

Reviewers

kamil
tomek

Summary

Added a hook that dispatches holder processing action as a result of DM ops.

Depends on D13407, D13409, D13410

Test Plan

Created dummy blob op in dm-ops/change-thread-settings for creating and removing a holder.
Watched Redux Devtools and Blob HTTP traffic to confirm that holders are established and removed correctly.
Also tested with D13413 test plan

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Fri, Sep 20, 6:30 AM

bartek held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptFri, Sep 20, 6:30 AM

bartek added a child revision: D13413: [lib] Create holder ops for thick thread avatars.Fri, Sep 20, 6:42 AM

bartek edited the test plan for this revision. (Show Details)Fri, Sep 20, 6:42 AM

Harbormaster failed remote builds in B31740: Diff 44374!Fri, Sep 20, 6:47 AM

Rebase

Harbormaster failed remote builds in B31744: Diff 44380!Fri, Sep 20, 7:05 AM

bartek published this revision for review.Fri, Sep 20, 7:09 AM

Harbormaster completed remote builds in B31747: Diff 44383.Fri, Sep 20, 7:17 AM

bartek added a child revision: D13416: [lib] Clear all holders on logout.Fri, Sep 20, 7:41 AM

ashoat added inline comments.Fri, Sep 20, 8:02 AM

lib/actions/holder-actions.js
141–142 ↗	(On Diff #44383)	Why do we want until `getAuthMetadata()` resolves before starting `createBlobHolderGenerator()`?

Revision Contents
Changeset List

Path

Size

lib/

actions/

holder-actions.js

81 lines

shared/

dm-ops/

process-dm-ops.js

32 lines

Diff 44374

View Options

lib/actions/holder-actions.js