mirror of
https://github.com/tzapu/WiFiManager.git
synced 2026-04-27 09:05:56 +03:00
[GH-ISSUE #126] Error with include order in docs #95
Labels
No labels
📶 WiFi
🕸️ HTTP
Branch
DEV Help Wanted
Discussion
Documentation
ESP32
Example
Good First Issue
Hotfix
In Progress
Incomplete
Needs Feeback
Priority
QA
Question
Task
Upstream/Dependancy
bug
duplicate
enhancement
invalid
pull-request
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/WiFiManager#95
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @SeanBannister on GitHub (Mar 15, 2016).
Original GitHub issue: https://github.com/tzapu/WiFiManager/issues/126
In README.md it lists the order of the includes as:
I had to change this to:
To avoid the following error:
error: 'WiFiClient' does not name a type
@tzapu commented on GitHub (Mar 15, 2016):
this is how i have it in all my sketches
not working for you ?
@SeanBannister commented on GitHub (Mar 15, 2016):
As soon as I change it to that I get the following errors (but by switching them it works fine):
@tzapu commented on GitHub (Mar 15, 2016):
what arduino ide are you using?
@SeanBannister commented on GitHub (Mar 15, 2016):
I was running Arduino IDE 1.6.7 so I upgraded to 1.6.8 but I'm still experiencing the same issue.
ESP8266 core for Arduino - 2.1.0
WifiManager - 0.10
Windows 8.1 Pro
@tzapu commented on GitHub (Mar 15, 2016):
hi, do you include ESP8266WiFi.h after those 3 includes when it errors?
@SeanBannister commented on GitHub (Mar 15, 2016):
No, but here's the simplest example that causes the error:
@tzapu commented on GitHub (Mar 15, 2016):
try adding this on top of all those
@SeanBannister commented on GitHub (Mar 15, 2016):
That works :) thanks for your help.
Maybe this could be added to the readme.md?
@tzapu commented on GitHub (Mar 15, 2016):
ok, thanks
i should probably add it then if it s not clear
cheers
@tzapu commented on GitHub (Mar 16, 2016):
i ve added this now, thanks