Page MenuHomePhabricator

[native] Introduce RegistrationContext
ClosedPublic

Authored by ashoat on Jun 8 2023, 6:52 AM.
Tags
None
Referenced Files
F3374675: D8148.id27552.diff
Tue, Nov 26, 4:42 PM
Unknown Object (File)
Fri, Nov 22, 9:48 PM
Unknown Object (File)
Mon, Nov 11, 1:57 AM
Unknown Object (File)
Oct 27 2024, 2:48 AM
Unknown Object (File)
Oct 25 2024, 5:30 PM
Unknown Object (File)
Oct 2 2024, 6:33 AM
Unknown Object (File)
Oct 2 2024, 6:33 AM
Unknown Object (File)
Oct 2 2024, 6:28 AM
Subscribers

Details

Summary

Explanation for why we need a context is in ENG-4047 and ENG-4043.

This diff just introduces an empty context; later diffs will actually implement functionality.

Test Plan

The whole stack was tested as follows:

  1. Try creating an account with the default avatar
  2. Try creating an account with an emoji avatar
  3. Try creating an account with an image avatar (that needs to be uploaded)
  4. Try creating an Ethereum account with an ENS avatar

I made sure of the following:

  1. Loading spinner appeared on the button, not on the avatar, and was present throughout
  2. After the RegistrationNavigator is dismissed, the avatar that appears for the user's PRIVATE chat in the ChatThreadList is correct
  3. Subsequent delete-account-and-then-register-again worked correctly
  4. Various error cases are triggered correctly

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage