Page MenuHomePhabricator

[native] handler that just logs out if missing CSAT and no password
AcceptedPublic

Authored by varun on Mon, May 6, 7:20 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 11, 12:02 PM
Unknown Object (File)
Fri, May 10, 1:00 PM
Subscribers
None

Details

Reviewers
ashoat
tomek
Summary

this is similar to the LogOutIfMissingCSATHandler but we only log out wallet users since we can't auth in the background for them

Test Plan

as a wallet user on native, with usingCommServicesAccessToken set to true, i cleared the CSAT from redux and confirmed that i was logged out

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

varun requested review of this revision.Mon, May 6, 7:36 AM
ashoat added inline comments.
native/components/background-identity-login-handler.react.js
14

Are we going with this name because this component will also handle automatically logging users in?

33

Slightly different name here than on web. Did you go with this because it's shorter?

34

This appears to be copy-pasted from web. Should we consider factoring it out, or do you think it's fine to copy-paste?

This revision is now accepted and ready to land.Mon, May 6, 11:56 AM