-
Notifications
You must be signed in to change notification settings - Fork 5
refactor launcher to use usb mouse library #70
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
Conversation
The similar PR to Minesweeper is adafruit/Adafruit_Learning_System_Guides#3118 |
Since we're requiring all apps to know about the override setting this needs to be updated or the boot animation will crash when used with an app that has overridden the volume. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this! a few changes noted in comments above
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! looks good.
Fixes #61
I'm noticing that the Minesweeper app is hitting a USB crash fairly frequently now, but I don't think that has anything to do with this update, I think it has to do with the improved error handling in the core.
I'm going to submit a similar PR to the learning system to convert minesweeper to using the adafruit_usb_host_mouse library and if necessary update the response to usb errors from the core.