Page MenuHomePhabricator
Differential D8049

[native] Default to ENS avatar for users that have one
ClosedPublic
Actions

Authored by ashoat on May 31 2023, 1:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jan 2, 9:49 AM
Unknown Object (File)
Thu, Jan 2, 8:25 AM
Unknown Object (File)
Sun, Dec 15, 1:45 PM
Unknown Object (File)
Sun, Dec 15, 1:45 PM
Unknown Object (File)
Sun, Dec 15, 1:42 PM
Unknown Object (File)
Sun, Dec 15, 1:19 PM
Unknown Object (File)
Nov 11 2024, 10:41 AM
Unknown Object (File)
Nov 11 2024, 4:50 AM
View All 68 Files
Subscribers
tomek

Details

Reviewers
atul
ginsu
Commits
rCOMMfd1cec1c4e0a: [native] Default to ENS avatar for users that have one
Summary

In the new registration flow, I want to default to using an ENS avatar if the user has one.

Depends on D8048

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

Since this is the last diff in the stack, I'll also include a video:

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
native/
account/
registration/
23 lines
25 lines
avatars/
9 lines

Diff 27368

View Options

native/account/registration/avatar-selection.react.js

Loading...
View Options

native/account/registration/connect-ethereum.react.js

Loading...
View Options

native/avatars/edit-user-avatar.react.js

Loading...