Page MenuHomePhabricator
Differential D12373

[lib] factor out broadcasting device list updates to hook
ClosedPublic
Actions

Authored by kamil on Jun 10 2024, 4:25 AM.
Tags
None
Referenced Files
F3382995: D12373.diff
Thu, Nov 28, 1:00 PM
Unknown Object (File)
Mon, Nov 25, 1:38 AM
Unknown Object (File)
Wed, Nov 20, 2:40 PM
Unknown Object (File)
Wed, Nov 20, 2:40 PM
Unknown Object (File)
Sat, Nov 2, 12:06 AM
Unknown Object (File)
Sat, Nov 2, 12:06 AM
Unknown Object (File)
Sat, Nov 2, 12:06 AM
Unknown Object (File)
Fri, Nov 1, 11:42 PM
View All 51 Files
Subscribers
ashoat

Details

Reviewers
bartek
tomek
Commits
rCOMMb1a04e641028: [lib] factor out broadcasting device list updates to hook
Summary

We want to reuse this code in more places, whenever disseminating device list updates.

Depends on D12347

Test Plan

Hook tested by calling function.
Secondary device auth not tested because of: ENG-8385.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil created this revision.Jun 10 2024, 4:25 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptJun 10 2024, 4:25 AM
kamil added a child revision: D12374: [lib] factor out getting and updating device list to hook.Jun 10 2024, 4:32 AM
Harbormaster completed remote builds in B29566: Diff 41153.Jun 10 2024, 4:40 AM
kamil edited the summary of this revision. (Show Details)Jun 10 2024, 9:41 AM
kamil published this revision for review.Jun 10 2024, 9:50 AM
tomek accepted this revision.Jun 12 2024, 5:43 AM
This revision is now accepted and ready to land.Jun 12 2024, 5:43 AM
bartek added inline comments.Jun 12 2024, 8:44 AM
native/profile/secondary-device-qr-code-scanner.react.js
107–112 ↗(On Diff #41153)

We have fetchLatestDeviceList() in lib/shared/device-list-utils.js that does exactly this

kamil added inline comments.Jun 13 2024, 7:54 AM
native/profile/secondary-device-qr-code-scanner.react.js
107–112 ↗(On Diff #41153)

yeah but in this case we also need lastSignedDeviceList

Closed by commit rCOMMb1a04e641028: [lib] factor out broadcasting device list updates to hook (authored by kamil). · Explain WhyJun 13 2024, 9:08 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMMb1a04e641028: [lib] factor out broadcasting device list updates to hook.

Revision Contents
Changeset List

PathSize
lib/
hooks/
50 lines
native/
profile/
50 lines

Diff 41298

View Options

lib/hooks/peer-list-hooks.js

Loading...
View Options

native/profile/secondary-device-qr-code-scanner.react.js

Loading...