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

Remove expired usernames from display on "choose accounts" screen #465

Closed
markmhendrickson opened this issue Jul 1, 2020 · 6 comments
Closed
Labels
area:bns enhancement-p3 Non-critical functionality wanted by many users, or there are clear alternatives

Comments

@markmhendrickson
Copy link
Collaborator

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

Screen Shot 2020-07-01 at 15.08.22.png

@jasperjansz jasperjansz removed their assignment Jul 24, 2020
@markmhendrickson markmhendrickson added bug Functionality broken bug-p2 Critical functionality broken for few users, with no clear workarounds sign-in labels Sep 23, 2020
@hstove hstove added enhancement-p3 Non-critical functionality wanted by many users, or there are clear alternatives effort:large Expected to take over 1 week of integration work labels Sep 23, 2020
@markmhendrickson markmhendrickson removed this from the 2020 W29-W31 milestone Sep 24, 2020
@aulneau
Copy link
Contributor

aulneau commented Mar 16, 2021

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

@markmhendrickson
Copy link
Collaborator Author

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 markmhendrickson changed the title Disable or hide expired usernames in account selection modal Indicate expired usernames in account selection modal with path to renewal Mar 17, 2021
@markmhendrickson markmhendrickson removed bug-p2 Critical functionality broken for few users, with no clear workarounds bug Functionality broken labels Mar 17, 2021
@aulneau
Copy link
Contributor

aulneau commented Mar 18, 2021

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.

@hstove
Copy link
Contributor

hstove commented Mar 26, 2021

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

@markmhendrickson markmhendrickson changed the title Indicate expired usernames in account selection modal with path to renewal Remove expired usernames from showing on home and choose accounts Apr 28, 2022
@markmhendrickson markmhendrickson removed the effort:large Expected to take over 1 week of integration work label Apr 28, 2022
@markmhendrickson markmhendrickson changed the title Remove expired usernames from showing on home and choose accounts Remove expired usernames from display on "choose accounts" screen Apr 28, 2022
@markmhendrickson
Copy link
Collaborator Author

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 #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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:bns enhancement-p3 Non-critical functionality wanted by many users, or there are clear alternatives
Projects
None yet
Development

No branches or pull requests

5 participants