-
Notifications
You must be signed in to change notification settings - Fork 31
Add Adafruit IO REST Client, Examples, and more! #1
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
…TPClient, add some docstrings
@brentru the weather example failes:
the randomizer test also fails with 404 |
@jerryneedell The |
got it -- working now |
How do I get the time correctly -- I think I got millis and seconds OK , but how do I print the ISO-8601 time?
it fails on the call to io.receive_time("ISO-8601") |
@jerryneedell Fixed in this commit.
|
great! downloaded and ran -- worked great! |
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.
I have tested all the examples via a feather nrf52840 and ESP32 (ESPRESIF devKit) -- all work fine!
The code looks good to me -- I think this is ready to merge and get more folks using it.
@jerryneedell thanks, just did a re-test on a fresh install of 4.0b2. Merging |
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.
So, this got merged while I was looking it over; makes sense as it was approved. 😄
But, wanted to make note of a couple things.
This does look like it was a fun one to work on!
Ready for public consumption and review!
Adding:
adafruit_io.py
: CircuitPython Adafruit IO REST Clientexamples
: Usage examples for the library (sensor communication, feed interaction, data, metadata)wifi_settings.py
, instead ofesp32_
to allow for an identicalWiFiManager
class in the Adafruit_CircuitPython_ESP_ATcontrol library.Examples tested on a PyPortal running CircuitPython 4 Beta 2 prior to PR'ing