-
Notifications
You must be signed in to change notification settings - Fork 12
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
feat: idOS design enhancement #607
Conversation
@shelegdmitriy a loom would be helpful. What is the expected UX when landing on Settings.Verifications.Index for a user who has already verified humanness? I'm setting the UI show the page as if I have not yet setup idos, the only option it gives is to start the setup flow again. ![]() |
Ah, looks like this PR must be merged before this UX can be tested. |
That's right |
@shelegdmitriy what's the expected UX here? I'm connected via mynearwallet, I'm already verified, but I get the UX as if I have never set this up. This is with bos-loader serving the components from this PR on beta.near.org which has your latest PR. ![]() |
Could you try to clear your local storage? |
@shelegdmitriy I experience the same when connecting on an incognito window. |
That's very interesting because it seems I found a couple of gaps in my code. Let me double check it and I'll ping you to test on your end. Sorry for that |
14dc4be
to
89060b4
Compare
This PR introduces design enhancements.
A couple of things I made different but I can record a loom video describing all the changes I've made.
Closes: #885
Related to near/near-discovery#913