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

Handle partial connections better #43

Open
2 of 5 tasks
makermelissa opened this issue Dec 4, 2022 · 0 comments
Open
2 of 5 tasks

Handle partial connections better #43

makermelissa opened this issue Dec 4, 2022 · 0 comments

Comments

@makermelissa
Copy link
Collaborator

makermelissa commented Dec 4, 2022

For USB and possibly BLE, we require the user to perform at least 2 steps to connect. If the user cancels in the middle of this, I'm not sure if this is being handled properly. Additionally, the UI could use some improvement to better inform the user that a step is complete.
I don't think this affects Web Workflow because it's basically 1 step, though I suppose if the user fails to type in credentials it may not be handled properly, so that should probably be checked too, though I imagine the result is they'll either get an error or it will prompt them again next time.

Here's what I think needs to be fixed for this issue to be complete:

  • If the user closes the dialog early, a disconnect should be performed
  • Check that Web Workflow is properly handled if the credentials dialog is dismissed
  • As the user completes steps, something like a checkmark should visually inform the user that it is complete
  • If the instructions are on a small screen like a phone, it should scroll to the next step the user is expected to do
  • Fix it if there is an issue so that the user knows what to do
@makermelissa makermelissa changed the title Handle partially connections better Handle partial connections better Dec 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant