Page MenuHomePhabricator

[native] introduce use ENS avatar to edit user avatar
ClosedPublic

Authored by ginsu on Apr 19 2023, 3:14 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 16, 5:21 AM
Unknown Object (File)
Mon, May 13, 8:25 PM
Unknown Object (File)
Thu, May 2, 10:57 PM
Unknown Object (File)
Mar 28 2024, 7:35 AM
Unknown Object (File)
Feb 29 2024, 2:31 AM
Unknown Object (File)
Feb 27 2024, 2:58 PM
Unknown Object (File)
Feb 27 2024, 2:58 PM
Unknown Object (File)
Feb 27 2024, 2:58 PM
Subscribers

Details

Summary

Add use ens avatar to the edit user avatar action sheet. This diff also introduces useENSUserAvatar to avatar-hooks which handles the logic for saving the ens avatar into our db.

I am aware I wrote useENSUserAvatar with a bit of a naive approach, but I think the goal should be to get things working first so we can get something shipped, then make things more elegant.

Test Plan

Followed the test plan from D7462 and was able to get all the same behavoir/results

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

native/avatars/avatar-hooks.js
357 ↗(On Diff #25426)

The structure of this hook is very similar to useRemoveUserAvatar (just above). Eventually we will factor out things in this file to reduce redundancy

Harbormaster returned this revision to the author for changes because remote builds failed.Apr 19 2023, 3:37 PM
Harbormaster failed remote builds in B18561: Diff 25426!
ginsu requested review of this revision.Apr 19 2023, 3:40 PM
ginsu edited the summary of this revision. (Show Details)

rebase in atuls changes

This revision is now accepted and ready to land.Apr 19 2023, 6:15 PM
This revision was landed with ongoing or failed builds.Apr 19 2023, 6:24 PM
This revision was automatically updated to reflect the committed changes.