Page MenuHomePhabricator
Differential D11842

[lib/web] add test button to create initial peer list
ClosedPublic
Actions

Authored by kamil on Tue, Apr 30, 6:53 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 15, 8:22 PM
Unknown Object (File)
Wed, May 15, 1:26 PM
Unknown Object (File)
Mon, May 13, 5:02 PM
Unknown Object (File)
Mon, May 13, 11:25 AM
Unknown Object (File)
Mon, May 13, 3:15 AM
Unknown Object (File)
Mon, May 13, 1:44 AM
Unknown Object (File)
Sun, May 12, 12:12 AM
Unknown Object (File)
Sat, May 11, 8:29 PM
View All 16 Files
Subscribers
ashoat

Details

Reviewers
tomek
bartek
Commits
rCOMM43341ddfa482: [lib/web] add test button to create initial peer list
Summary

Before Identity launch, there isn't make any sense to run this migration, right now adding this only for testing purposes but the code can be reused.

Depends on D11841

Test Plan
  1. Test is store was properly populated
  2. Test is persisting works

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil created this revision.Tue, Apr 30, 6:53 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptTue, Apr 30, 6:53 AM
kamil added a child revision: D11843: [native] add test button to create initial peer list.Tue, Apr 30, 6:54 AM
Harbormaster completed remote builds in B28601: Diff 39694.Tue, Apr 30, 7:10 AM
kamil published this revision for review.Mon, May 6, 4:25 AM
kamil edited the summary of this revision. (Show Details)
kamil added 1 blocking reviewer(s): bartek.
ashoat mentioned this in D11843: [native] add test button to create initial peer list.Mon, May 6, 11:43 AM
bartek accepted this revision.Tue, May 7, 1:44 AM
bartek added inline comments.
lib/utils/device-list-utils.js
15 ↗(On Diff #39694)

Wondering if we should add a validator for RawDeviceList and assert it here

This revision is now accepted and ready to land.Tue, May 7, 1:44 AM
kamil updated this revision to Diff 39881.Tue, May 7, 4:17 AM

add assert & validator

Harbormaster completed remote builds in B28727: Diff 39881.Tue, May 7, 4:36 AM
Closed by commit rCOMM43341ddfa482: [lib/web] add test button to create initial peer list (authored by kamil). · Explain WhyTue, May 7, 4:55 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMM43341ddfa482: [lib/web] add test button to create initial peer list.

Revision Contents
Changeset List

PathSize
lib/
selectors/
4 lines
types/
5 lines
utils/
26 lines
web/
settings/
34 lines

Diff 39887

View Options

lib/selectors/user-selectors.js

Loading...
View Options

lib/types/identity-service-types.js

Loading...
View Options

lib/utils/device-list-utils.js

Loading...
View Options

web/settings/account-settings.react.js

Loading...