Page MenuHomePhabricator
Differential D13972

[lib] standarize broadcasting device list update when removing device
ClosedPublic
Actions

Authored by kamil on Nov 20 2024, 3:05 AM.
Tags
None
Referenced Files
F3513493: D13972.id45894.diff
Sun, Dec 22, 12:25 AM
F3513441: D13972.id.diff
Sun, Dec 22, 12:24 AM
F3513429: D13972.diff
Sun, Dec 22, 12:24 AM
Unknown Object (File)
Mon, Dec 16, 3:35 PM
Unknown Object (File)
Mon, Dec 16, 11:33 AM
Unknown Object (File)
Mon, Dec 16, 2:42 AM
Unknown Object (File)
Mon, Dec 16, 2:42 AM
Unknown Object (File)
Sun, Dec 15, 8:07 PM
View All 26 Files
Subscribers
ashoat
tomek

Details

Reviewers
bartek
ashoat
Commits
rCOMMfb55cfff0e59: [lib] standarize broadcasting device list update when removing device
Summary

Addressing https://phab.comm.dev/D13963#inline-78590 and https://phab.comm.dev/D13963#inline-78595.

Right now we have the same approach each time:

  1. Sending device list updates to all our peers
  2. Sending device list updates to all our devices that are on the device list, but without the primary device
  3. Call getAndUpdateDeviceListsForUsers for our userID
    • update AuxUserStore
    • get platform details if the device was added

Depends on D13963

Test Plan

Remove secondary device and test if other peers received device list update

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil created this revision.Nov 20 2024, 3:05 AM
kamil held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptNov 20 2024, 3:05 AM
kamil edited the summary of this revision. (Show Details)Nov 20 2024, 3:08 AM
kamil mentioned this in D13963: [lib] create hook to run Device List Update Protocol.
kamil published this revision for review.Nov 20 2024, 3:10 AM
kamil edited the summary of this revision. (Show Details)
Harbormaster completed remote builds in B32722: Diff 45894.Nov 20 2024, 3:23 AM
ashoat added inline comments.Nov 20 2024, 4:23 AM
lib/shared/device-list-utils.js
345 ↗(On Diff #45894)

We could consider extracting this outside the if branches since it's shared across all branches

356 ↗(On Diff #45894)

Same with this line

ashoat accepted this revision.Nov 20 2024, 4:23 AM
This revision is now accepted and ready to land.Nov 20 2024, 4:23 AM
kamil updated this revision to Diff 45987.Mon, Nov 25, 2:23 AM

address review

Harbormaster completed remote builds in B32800: Diff 45987.Mon, Nov 25, 2:40 AM
Closed by commit rCOMMfb55cfff0e59: [lib] standarize broadcasting device list update when removing device (authored by kamil). · Explain WhyMon, Nov 25, 4:06 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMMfb55cfff0e59: [lib] standarize broadcasting device list update when removing device.

Revision Contents
Changeset List

PathSize
lib/
shared/
57 lines

Diff 46000

View Options

lib/shared/device-list-utils.js

Loading...