extension icon indicating copy to clipboard operation
extension copied to clipboard

Remove expired usernames from display on "choose accounts" screen

Open markmhendrickson opened this issue 5 years ago • 6 comments

Perhaps adding a link to documentation about renewing or re-buying expired usernames.

Screen Shot 2020-07-01 at 15.08.22.png

markmhendrickson avatar Jul 01 '20 13:07 markmhendrickson

https://www.figma.com/file/aXnNi8MOmN3CDrVwaDXXuqX3/Authenticator?node-id=7829%3A89907

jasperjansz avatar Jul 24 '20 11:07 jasperjansz

I'm not sure hiding or disabling them would be the right course to choose. Even if a username is expired, the account could still use apps that don't require them

aulneau avatar Mar 16 '21 19:03 aulneau

That's a good point. Perhaps we should flag that expiration and provide a path for renewal, but otherwise allow the user to authenticate with just the associated address.

markmhendrickson avatar Mar 17 '21 11:03 markmhendrickson

otherwise allow the user to authenticate with just the associated address.

just to clarify, regardless of if an account has a username, the associated address is what is always authenticating.

aulneau avatar Mar 18 '21 20:03 aulneau

just to clarify, regardless of if an account has a username, the associated address is what is always authenticating.

👍🏼 I think the main issue is that @stacks/auth will fail if an expired username is included in the authResponse, so simply being smarter about that (and not including the username) is the main impact

hstove avatar Mar 26 '21 16:03 hstove

Expired names should no longer impede the selection of accounts for app authentication now that @janniks has removed username-checking from auth in general with https://github.com/hirosystems/stacks-wallet-web/pull/2333.

However, these expired names still do appear on the "choose accounts" screen. I've renamed this issue to focus on their removal from there, reverting back to "Account N" just as how things work on the home screen as desired:

image image

markmhendrickson avatar Apr 28 '22 11:04 markmhendrickson