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

Completely revamped flashing flow #92

Merged
merged 1 commit into from Dec 22, 2018
Merged

Conversation

algernon
Copy link
Collaborator

When clicking the "Upload" button on the flashing card, instead of flashing right away, a dialog will pop up with some explanatory text about what will happen, and what to expect. This fixes #83.

After flashing is done, we'll fall back to the keyboard selection screen, because reconnecting is not reliable yet. This should fix the white screen after flashing, and as such, this also closes #69.

screenshot from 2018-12-22 01-26-31

When clicking the "Upload" button on the flashing card, instead of flashing
right away, a dialog will pop up with some explanatory text about what will
happen, and what to expect. This fixes #83.

After flashing is done, we'll fall back to the keyboard selection screen,
because reconnecting is not reliable yet. This should fix the white screen after
flashing, and as such, this also closes #69.

Signed-off-by: Gergely Nagy <algernon@keyboard.io>
@algernon algernon added bug Something isn't working enhancement New feature or request pending labels Dec 22, 2018
@algernon algernon added this to the Chrysalis 0.1.0 milestone Dec 22, 2018
@algernon algernon requested a review from obra December 22, 2018 00:27
@algernon
Copy link
Collaborator Author

Flashing is still a tad flaky, because we're closing the serial port, and opening the same one again. Ideally, if we had it open, we'd just use it.

That's a separate issue, however, and one that needs deeper looking into.

@obra obra merged commit 1ad79bf into master Dec 22, 2018
@obra
Copy link
Member

obra commented Dec 22, 2018

<3 <3

@obra obra deleted the flasher/press-prog-and-play branch December 22, 2018 01:31
0957758592 referenced this pull request in Dygmalab/Bazecor Nov 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
2 participants