-
Notifications
You must be signed in to change notification settings - Fork 108
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
Async validation #92
Comments
Sounds reasonable. I'll have a looksy later. |
Follow-up to keep you in the loop: I started looking into this, but quickly realized we need to provide feedback to the user while waiting. Not entirely obvious how that would fit into the current implementation flow. Asked for input, but haven't gotten any, yet. |
Thx. I see your point. It is already worth having it without a spinner IMO. Most validation flows should be quick (db calls are in the ms). But I understand in case you want to have it perfect. |
Progress has been made. Have some rough edges to smooth out. |
How/where could we enjoy this progress? :) |
Is your feature request related to a problem? Please describe.
I ask the user for an account id. I like to validate this id by checking it against the db which I can't do sync.
Describe the solution you'd like
Make
validate
compatible withasync
.Describe alternatives you've considered
Currently I load all the accounts before the prompt so I can validate the id sync. This doesn't scale though.
Additional context
That's it :)
The text was updated successfully, but these errors were encountered: