-
Notifications
You must be signed in to change notification settings - Fork 771
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
update BLE remote control switch; not tested #937
base: main
Are you sure you want to change the base?
update BLE remote control switch; not tested #937
Conversation
80a9f38
to
a8eae25
Compare
@rdagger do you have time to review this, and still have the hardware set up? Procedure would be to:
Thanks. |
Did a quick test: Client code through the following error: The server code didn't throw an error but when I tried to connect with the Bluefruit Connect app, I received the following error on my phone: I'm using Adafruit CircuitPython 5.0.0-beta.0-45-gd154d1191 on 2019-11-25; Adafruit Feather nRF52840 Express with nRF52840 and the latest version of CircuitPython Bundle and Adafruit_BLE. |
@rdagger OK, I fixed some code in client.py. I am not getting an error on server.py: it connects fine from my iPhone 6S running the latest OS and app version. What phone are you using? |
@dhalbert Google Nexus 6 |
Charging up a Samsung Android tablet to check. |
I connected with Samsung Tablet, though I had a failed connection first, and then reset the Feather, and it worked second time. Not sure what that was about (maybe leftover connection from iOS). |
@dhalbert I restarted my Android phone and the server code is working now using the Bluefruit phone app. |
@dhalbert I spoke too soon. After a few minutes I received the following error from the server code: Traceback (most recent call last): |
it's on my list to rewrite. :) |
@dhalbert - is this PR pertinent given the latest work? I'm looking to close open Learn PRs. Thanks |
This updates the code for https://learn.adafruit.com/circuitpython-ble-remote-control-on-off. Needs testing by @rdagger.