Follow-up to the previous diff that combines isLoggedIn. This lets us factor out some logic in the codebase.
Depends on D12696
Details
Details
This should be a simple no-op refactor, so my testing was limited:
- Flow
- Careful inspection of the code
- The changes to NavigationHandler were tested as part of the stack
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| native/components/auto-join-community-handler.react.js | ||
|---|---|---|
| 36 ↗ | (On Diff #42122) | There is a minor change here and in ConnectFarcasterAlertHandler below. Previously currentUserID could be set for an anonymous cookie, but now that's not possible. I think this a good change – neither of these handlers should be called if the cookie is anonymous |