safe-react icon indicating copy to clipboard operation
safe-react copied to clipboard

Allow selecting an NFT as a Safe profile pic

Open katspaugh opened this issue 2 years ago • 3 comments

Overview

We're currently displaying a randomly generated pic for all Safes. Some users might want to more easily differentiate their NFT-holding Safes by looking at the profile pic.

Goals

Allow users to set a specific NFT held in the Safe as this Safe's profile pic.

Requirements

  • On the Collectibles page, allow people to select an NFT to be a profile pic
  • Store the selection in the localStorage as a hashmap of addresses to image URLs
  • Display the profile pic in the Safe info
  • Add a "Select a profile picture" link to Collectibles from the Appearance settings

Screens

Safe info Screenshot 2022-01-03 at 09 12 44 -> Screenshot 2022-01-03 at 09 18 13

Collectibles page Screenshot 2022-01-03 at 09 26 29

katspaugh avatar Jan 03 '22 08:01 katspaugh

Interesting idea for sure. Would you assume this profile picture to also be displayed for other users or just the user that set it (i.e. have this information stored locally or also exchanged via backend or even on-chain)?

For an on-chain solutions we could look at ENS which allows to set a profile picture for a ENS domain, combined a stronger integration of reverse resolving this could go into a similar direction. However, ENS profile picture do not contain proof of ownership which would be an interesting property when setting a profile picture from an NFT held in a Safe.

lukasschor avatar Jan 03 '22 08:01 lukasschor

I meant just locally, similar how we can give a Safe a name. Good idea about grabbing that from ENS, I'll update the ticket.

katspaugh avatar Jan 03 '22 09:01 katspaugh

Unstoppable domains also allow to set an NFT as a profile picture

mmv08 avatar Jan 03 '22 10:01 mmv08