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

Authored by varun on May 6 2024, 7:20 AM.

Details

Reviewers

ashoat
tomek

Commits

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

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

varun requested review of this revision.May 6 2024, 7:36 AM

ashoat added inline comments.

native/components/background-identity-login-handler.react.js
14 ↗	(On Diff #39850)	Are we going with this name because this component will also handle automatically logging users in?
33 ↗	(On Diff #39850)	Slightly different name here than on `web`. Did you go with this because it's shorter?
34 ↗	(On Diff #39850)	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.May 6 2024, 11:56 AM

native/components/background-identity-login-handler.react.js
14 ↗	(On Diff #39850)	yeah. should have the diff up soon that adds the auto login logic to this component
33 ↗	(On Diff #39850)	Yeah, in the alert it looked better with less text
34 ↗	(On Diff #39850)	i can factor it out before landing