Page MenuHomePhabricator

[native] Construct synthetic UserInfo to pass to EditUserAvatar
ClosedPublic

Authored by ashoat on May 31 2023, 1:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 17, 8:36 AM
Unknown Object (File)
Thu, Apr 4, 9:10 PM
Unknown Object (File)
Thu, Apr 4, 9:10 PM
Unknown Object (File)
Thu, Apr 4, 9:09 PM
Unknown Object (File)
Thu, Apr 4, 9:01 PM
Unknown Object (File)
Feb 11 2024, 12:20 AM
Unknown Object (File)
Feb 11 2024, 12:08 AM
Unknown Object (File)
Feb 11 2024, 12:08 AM
Subscribers

Details

Summary

EditUserAvatar renders an avatar via UserAvatar, which currently only works when the data (UserInfo or CurrentUserInfo) is in Redux.

For the registration flow, we want to render a "pending" avatar that hasn't been saved into Redux yet. To that end, this diff introduces a way to pass UserInfo to UserAvatar and EditUserAvatar.

Depends on D8045

Test Plan

This was tested in combination with the rest of the stack. I tested:

  1. ENS flow, both when user has an ENS avatar and when they don't
  2. Setting an emoji avatar
  3. Setting an image avatar via camera
  4. Setting an image avatar via media library
  5. Tested avatar update flow after being logged in to make sure I didn't regress anything

To see the UI in action, check out the video in D8049!

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable