Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MVP 2 ens username in chat #7114

Closed
yenda opened this issue Dec 17, 2018 · 13 comments
Closed

MVP 2 ens username in chat #7114

yenda opened this issue Dec 17, 2018 · 13 comments

Comments

@yenda
Copy link
Contributor

yenda commented Dec 17, 2018

Problem

As a user, I want my ens username to be displayed in chats I participate to.
As a user, I need to be able to specify my ens username.
As a user, I need to be able to distinguish simple usernames from ens usernames.

Implementation

  • ens username is sent with the messages
  • if the user was previously unknown, the ens username is verified against the contract

Acceptance Criteria

MVP

  • ens usernames replace username in chats

An issue is created for 100% feature completion with following criteria

There is two edge cases are not accounted for in the MVP:

  • what should happen if I release my ens username?
  • what should happen if I decide to use a new ens username?
@yenda
Copy link
Contributor Author

yenda commented Dec 17, 2018

@andmironov is there any designs already regarding the user stories?
for this feature to work out we need to be able to distinguish username from ens usernames in chat, and the user needs to be able to set his ens-username somewhere

@andmironov
Copy link

cc @errorists
I wonder if it should be display name shown if specified, even if the user has a ENS username?

@yenda
Copy link
Contributor Author

yenda commented Dec 17, 2018

The simplest solution would be to merge the two, one could specify his ENS username as display name. We could add a "show in public chats" checkbox. The ens username would simply be a "verified" display name

@andmironov
Copy link

We should be really thoughtful and consistent about which name is used where, since the hierarchy is not 100% transparent.

  • Every user has an account name (e.g Attractive Salty Seal). Used as a human-readable contact code
  • Some users have a display name. May or may not be specified by user. May or may not be revealed to other users
  • Some users have a ENS username. Used as a unique user identifier.

Here are the new profile designs with and without those specified:

p

Questions i dont have answers to:

  • If a ENS username exists it might replace the account name ?
  • How the user should be shown in chat?

@yenda
Copy link
Contributor Author

yenda commented Dec 17, 2018

also a user can have multiple ens usernames, but for those who already registered one, the app can't know it, the user have to enter them somewhere

@andmironov
Copy link

andmironov commented Dec 17, 2018

also a user can have multiple ens usernames

this complicates this even more :-) please let's not rush into implementation, this requires some time for UX research. There are not too many apps with such a complex identity structure

cc @hesterbruikman

@yenda yenda changed the title MVP ens username in chat MVP 2 ens username in chat Dec 17, 2018
@rachelhamlin
Copy link
Contributor

I'd suggest we use ENS names only for default display in chat/profile—it increases the utility of the username.

Stateofus.eth names should have preferential treatment, displaying in chat as Rachel and not Rachel.stateofus.eth. Other ENS names could present as rachel.otherdomain.eth (if and when there is a time that other ENS names have Whisper IDs associated).

If at some point ENS registration is built into onboarding (potentially even for free), we could do away with non-unique display names entirely. This has the double benefit of making impersonation more difficult.

What we want to enable with this feature right now is this:

  • As a user with an ENS name with Whisper ID associated, I want the option to set my name to public by default, so that anyone can see my ENS name in chat and on my profile.
  • As a user with an ENS name without Whisper ID associated, I want to be offered the option to edit my name and associate my Whisper ID (via the ENS DApp*), so that I can then set it to be public.

Following that:

  • As a user who is displaying my ENS name publicly, I want the option to revert back to using my anon name by default.

Is that clear? I'll respond to the edge cases raised by @yenda in a subsequent post.

*This update flow exists in the ENS DApp, we'd just need to link to it.

@rachelhamlin
Copy link
Contributor

Also side note: This is somewhat required to make Tribute to Talk useful. Without it, you're never going to be able to identify people you want to contact (if Attractive Salty Seal is Brad Pitt, he can just hide that from me forever by not adding me as a contact, and I'm not going to pay tribute to Attractive Salty Seal).

@rachelhamlin
Copy link
Contributor

@yenda

also a user can have multiple ens usernames, but for those who already registered one, the app can't know it, the user have to enter them somewhere

On this, can the app detect any of a user's names, or does it require manual input?

@hesterbruikman
Copy link
Contributor

@andmironov fyi

This implies that an existing ENS username needs to be selectable as Display name in the new Profile screens.

@yenda
Copy link
Contributor Author

yenda commented Jan 9, 2019

@yenda

also a user can have multiple ens usernames, but for those who already registered one, the app can't know it, the user have to enter them somewhere

On this, can the app detect any of a user's names, or does it require manual input?

it requires manual input that can then be validated by the app

@rachelhamlin
Copy link
Contributor

@andmironov @yenda @hesterbruikman maybe we can hop on a call this week to talk about our options here?

@hesterbruikman
Copy link
Contributor

@rachelhamlin Following up on Status today

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants