-
Notifications
You must be signed in to change notification settings - Fork 93
Remove NTPClient used in SAMD examples #6
Comments
I am still a bit torn on this. I understand wanting to make the samples/sketches as concise as possible. However, this type of functionality does seem out of scope for the the library itself. While NTP and Wifi are requirements for using the library successfully it feels odd to move the configuration of them into the library. Should these libraries want to expose new initial configuration options then this library would also need to be updated instead of just benefiting from new functionality. |
@obsoleted good points, I'll submit a patch to this repo to also support non-ESP8266 boards: https://github.com/FWeinb/NTPClient Then we can use it in the examples like you suggested. I'll rename this issue accordingly. |
I've submitted the following PR to the NTPClient repo: arduino-libraries/NTPClient#8 |
Closed via #16. |
No description provided.
The text was updated successfully, but these errors were encountered: