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

Wifi manager frank cohen #1642

Merged
merged 4 commits into from Jan 20, 2024
Merged

Conversation

frankcohen
Copy link
Contributor

Hi, thanks for the great library. Here's a tiny contribution in return. I tested on ESP32-S3 so the readme now has the shield for ESP32-S3. And I fixed the Release shield from clicking to the image page. -Frank

Tested on ESP32-S3, so I added the ESP32-S3 shield to the readme
Added ESP32-S3 tag and fixed the release shield to not click away from the project.
@@ -5,7 +5,8 @@ Espressif ESPx WiFi Connection manager with fallback web configuration portal

:warning: This Documentation is out of date, see notes below

![Release](https://img.shields.io/github/v/release/tzapu/WiFiManager?include_prereleases)
<a name="release"></a>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<a name="release"></a>
<a name="release"></a>

@@ -5,7 +5,8 @@ Espressif ESPx WiFi Connection manager with fallback web configuration portal

:warning: This Documentation is out of date, see notes below

![Release](https://img.shields.io/github/v/release/tzapu/WiFiManager?include_prereleases)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
![Release](https://img.shields.io/github/v/release/tzapu/WiFiManager?include_prereleases)
![Release](https://img.shields.io/github/v/release/tzapu/WiFiManager?include_prereleases)

@tablatronix tablatronix merged commit 0d84861 into tzapu:master Jan 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants