Page MenuHomePhabricator

[native] Introduce RegistrationContext
ClosedPublic

Authored by ashoat on Jun 8 2023, 6:52 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 18, 2:21 AM
Unknown Object (File)
Thu, Apr 18, 2:21 AM
Unknown Object (File)
Thu, Apr 18, 2:21 AM
Unknown Object (File)
Thu, Apr 18, 2:17 AM
Unknown Object (File)
Feb 23 2024, 6:40 PM
Unknown Object (File)
Feb 23 2024, 6:02 PM
Unknown Object (File)
Feb 23 2024, 4:07 PM
Unknown Object (File)
Feb 23 2024, 4:00 PM
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
Lint Not Applicable
Unit
Tests Not Applicable