Skip to content

Conversation

@TimothyFran
Copy link
Contributor

Added WiFi.setAutoReconnect(true) inside _connect

@github-actions
Copy link
Contributor

github-actions bot commented Jan 13, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@TimothyFran
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Jan 13, 2025
@ayushsharma82
Copy link
Owner

ayushsharma82 commented Feb 19, 2025

Hi @TimothyFran , Thanks for the PR. If you look at this part of the code, WiFi.autoReconnect is disabled before _connect to avoid any rogue connection to STA while the device checks for it during initial setup.

I'd like to know more about the issue you faced. A better solution will be to modify the _connect function to have a third boolean parameter by which we can control when we need the auto-reconnect functionality.

@TimothyFran
Copy link
Contributor Author

TimothyFran commented Feb 19, 2025

The problem I'm facing is that if the Wi-Fi connection is lost it is not automatically reconnected once it is available again.

I'll update my changes to add a Boolean parameter as you suggested

@ayushsharma82 ayushsharma82 merged commit 7d72b88 into ayushsharma82:dev Feb 20, 2025
12 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Feb 20, 2025
@TimothyFran TimothyFran deleted the dev branch February 20, 2025 14:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants