-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Add ability to enter desired BSSID #4429
base: main
Are you sure you want to change the base?
Conversation
- add event handling (debug) - fixes Aircoookie#2151
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.
Good change overall with a nice move of initEthernet from wled.cpp to network.cpp.
Although most compilations (jobs) are successful, some of them have warnings (not breaking though)
Some of them seems to be related to your patch / code move.
That's just a harbinger of things to come. 😉 |
Please update the PR title to fully summarise the changes contained and extend the description as well please @blazoncek |
@netmindz the gist of this PR is in its title - ability to force BSSID selection. All other changes are just collateral damage from my fork. I do not intend to remove those as it is too much hassle otherwise. EDIT: The bugging part is a joke. |
@blazoncek I do not have a mesh system but to test this I had to use two esp32 nat router with the same SSID . One minor issue during the setup ( this issue has no impact on functionality ) is that the BSSID is populated with the SSIDs and one will be selected for you , once you hit save you will get a bad BSSID as you see in the pic below but as you have to enter that manually in any case then this is a minor thing but for sure will be confusing to others . |
Thank you. |
@blazoncek could you be a bit more specific? The file is not modified by your PR. |
@softhack007 it is unrelated to this PR. Will include more info when ready. |
WLED::initEthernet()
andWLED::findWiFi()
tonetwork.cpp
WLED_WIFI_CONFIGURED
with function call (and updated macro)WLED_ETHERNET
Oh, and BTW, there is a bug in
src/dependencies/network/Network.cpp
.