Page MenuHomePhabricator

[lib][web][native] add Tunnelbroker device token to redux
ClosedPublic

Authored by kamil on Jun 20 2024, 3:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Oct 22, 1:17 PM
Unknown Object (File)
Tue, Oct 22, 8:35 AM
Unknown Object (File)
Fri, Oct 18, 7:32 PM
Unknown Object (File)
Wed, Oct 16, 10:03 PM
Unknown Object (File)
Wed, Oct 16, 4:12 PM
Unknown Object (File)
Sep 28 2024, 5:16 AM
Unknown Object (File)
Sep 28 2024, 3:26 AM
Unknown Object (File)
Sep 25 2024, 9:33 PM
Subscribers

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.Jun 20 2024, 4:13 PM
lib/reducers/tunnelbroker-device-token-reducer.js
7–12 ↗(On Diff #41585)

I'm guessing this will be updated?

native/redux/persist.js
1424 ↗(On Diff #41585)

Don't we want to update this?

web/redux/persist.js
571 ↗(On Diff #41585)

Shouldn't we update storeVersion?

bump version

lib/reducers/tunnelbroker-device-token-reducer.js
7–12 ↗(On Diff #41585)

sure, in next diffs

native/redux/persist.js
1424 ↗(On Diff #41585)

yeah we want, forgot to commit that change, thanks for catching

This revision is now accepted and ready to land.Jun 21 2024, 2:35 AM