-
Notifications
You must be signed in to change notification settings - Fork 2.2k
[WiFi] Cleanup code, split ESP32/ESP8266 specific #5149
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
base: mega
Are you sure you want to change the base?
Conversation
|
Notes/bookmarks: AP API: https://github.com/espressif/arduino-esp32/blob/master/libraries/WiFi/src/WiFiAP.h#L44-L57 Manager api:
|
src/src/Globals/NWPlugins.cpp
Outdated
|
|
||
| // TODO TD-er: Implement | ||
|
|
||
| for (networkIndex_t x = 0; x < CONTROLLER_MAX; x++) { |
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.
Shouldn't that be NETWORK_MAX ?
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.
Yes it should...
…o bugfix/WiFi_state_machine
Finally going to clean up the WiFi code and restructure it.
Hopefully this will also show where the long lasting WiFi reconnect issues may be...
Fixes: #3870
ToDo:
WiFi.softAPgetStationNum()does not work as expected on ESP8266)WiFi_AP_Candidates.force_reload()when saving security settings"Sensors":[]missing)