Page MenuHomePhabricator

[lib][web][native] Introduce commServicesAccessToken to Redux state
ClosedPublic

Authored by ashoat on Mar 27 2023, 8:11 PM.
Tags
None
Referenced Files
F3525141: D7212.diff
Mon, Dec 23, 3:51 PM
Unknown Object (File)
Mon, Dec 16, 12:08 PM
Unknown Object (File)
Mon, Dec 16, 12:08 PM
Unknown Object (File)
Mon, Dec 16, 12:08 PM
Unknown Object (File)
Mon, Dec 16, 12:07 PM
Unknown Object (File)
Mon, Dec 16, 12:04 PM
Unknown Object (File)
Mon, Dec 9, 8:08 AM
Unknown Object (File)
Fri, Dec 6, 9:53 PM
Subscribers

Details

Summary

We'll use this to track the access token used to authenticate the client to Comm services.

Test Plan

I just tested this with Flow for now, but I'll test it more thoroughly in combination with the following diffs, and will wait to land it until then

Diff Detail

Repository
rCOMM Comm
Branch
ashoat/token
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

Add to redux-persist allowlist on native

LGTM, ill defer to react/redux people for correctness

Might be missing something, but should we add a migration to native that adds commServicesAccessToken field to Redux if it doesn't already exist?

This revision is now accepted and ready to land.Mar 28 2023, 1:27 PM
In D7212#214878, @atul wrote:

Might be missing something, but should we add a migration to native that adds commServicesAccessToken field to Redux if it doesn't already exist?

No reason to do that, I think... it would just set commServicesAccessToken to null, but state.commServicesAccessToken already returns undefined...

There will be follow-up diffs here for the reducer etc., though