Page MenuHomePhabricator

[lib] Introduce new SET_DATA_LOADED action
ClosedPublic

Authored by ashoat on Jun 6 2023, 8:19 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 4, 8:26 PM
Unknown Object (File)
Thu, Apr 4, 8:26 PM
Unknown Object (File)
Thu, Apr 4, 8:26 PM
Unknown Object (File)
Thu, Apr 4, 8:25 PM
Unknown Object (File)
Mar 4 2024, 11:30 PM
Unknown Object (File)
Mar 4 2024, 11:30 PM
Unknown Object (File)
Mar 4 2024, 3:04 AM
Unknown Object (File)
Mar 4 2024, 3:03 AM
Subscribers

Details

Summary

This is the first part of ENG-4048.

Context: dataLoaded is used in isLoggedIn here, which is in turn used in LogInHandler here to decide when to dismiss LoggedOutModal and the other accountModals (such as RegistrationNavigator).

We want to defer dismissal until the avatar setting keyserver endpoint returns. The easiest way to do that is make sure dataLoaded doesn't get set at the end of register. To achieve that, I'm going to use a separate Redux action to set dataLoaded, and in a later diff will stop registration from setting dataLoaded.

Test Plan

Tested registration (SIWE and otherwise) flows on native and web

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable