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

Update ESP32 and core libraries #50

Merged
merged 10 commits into from
Feb 3, 2024
Merged

Conversation

bitrot-alpha
Copy link
Contributor

This should fix issue #46

I tried to make as little changes as possible, aside from some whitespace and getting rid of extraneous block comments.
I've only tested this with the clock I have, the Elekstube IPS.

This version should work on Arduino IDE version 1.8.19 or 1.8.13.
I've compiled this against ESP32 Arduino version 2.0.9 (latest available as of date)

Library versions (see updated README.md)

  • NTPClient by Fabrice Weinberg (developed on v3.2.0) - 3.2.1 OK
  • Adafruit NeoPixel by Adafruit (developed on v1.8.0) - Now on 1.11.0
  • DS1307RTC by Michael Margolis (developed on v1.4.1)
  • TFT_eSPI by Bodmer (developed on v2.3.61) - 2.5.0 OK
  • Time by Michael Margolis (developed on v1.6.0) - 1.6.1 OK
  • PubSubClient by Nick O'Leary (developed on v2.8.0) https://www.arduinolibraries.info/libraries/pub-sub-client
  • ArduinoJson by Benoit Blanchon (worked on 6.19.4) - 6.21.2 OK

I've added some more "precompiled" clock faces in the my_data folder because I liked the look of the default nixie tube face that comes on the default firmware of the Elekstube.

@bitrot-alpha
Copy link
Contributor Author

Currently (on commit 44e62fa) I'm getting a crash right after the ESP32 finishes with WPS. It doesn't seem to harm anything though, and after it resets itself, it boots up and connects to WiFi fine.

@bitrot-alpha
Copy link
Contributor Author

WPS crash resolved. Return value of WiFiStartWPS is never set nor used, so I changed it to void.

@bitrot-alpha bitrot-alpha mentioned this pull request May 29, 2023
@aly-fly aly-fly added the enhancement New feature or request label Jan 18, 2024
@aly-fly aly-fly self-assigned this Feb 3, 2024
@aly-fly
Copy link
Owner

aly-fly commented Feb 3, 2024

Thank you @bitrot-alpha for your contributions. I have copied your changes into my repository (currently serves as a sandbox for testing all new additions). Files in this repo will be overwritten later.

@aly-fly aly-fly merged commit 414aa7f into aly-fly:main Feb 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants