[lib] Broadcast device list update during primary logout
Needs ReviewPublic
Actions

Authored by bartek on Fri, Jun 28, 2:18 AM.

Details

Reviewers

Summary

Calling LogOutPrimaryDevice RPC performs device list update on Identity. After this is done (but before local Redux and DB is cleared) we need to broadcast new device list to peers.

Depends on D12604

Test Plan

Tested it on natvie and web - the update is broadcasted despite being already logged out

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Fri, Jun 28, 2:18 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptFri, Jun 28, 2:18 AM

Harbormaster completed remote builds in B29982: Diff 41781.Fri, Jun 28, 2:34 AM

Did you forget to submit this one?

lib/actions/user-actions.js
265–266	This seems like a risky assumption. How do you know it's safe?

bartek added a child revision: D12636: [lib] Add Primary Device Logout P2P message type.Tue, Jul 2, 5:02 AM

bartek published this revision for review.Tue, Jul 2, 5:57 AM

bartek edited the test plan for this revision. (Show Details)

Revision Contents
Changeset List

Path

Size

lib/

actions/

user-actions.js

19 lines

Diff 41781

View Options

lib/actions/user-actions.js

[lib] Broadcast device list update during primary logoutNeeds ReviewPublicActions