mirror of
https://github.com/tzapu/WiFiManager.git
synced 2026-04-27 00:55:52 +03:00
[GH-ISSUE #968] dublicate function: /0wifi and /wifi&refresh #822
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#822
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 @dontsovcmc on GitHub (Nov 15, 2019).
Original GitHub issue: https://github.com/tzapu/WiFiManager/issues/968
If better to remove dublicate functions to decrease code size. I think /0wifi is excess, cause we can change button code:
"<form action='/wifi?refresh=1' method='get'><button>Configure WiFi</button></form><br/>\n", // MENU_WIFIno scan will be /wifi without argument:
"<form action='/wifi' method='get'><button>Configure WiFi (No Scan)</button></form><br/>\n", // MENU_WIFINOSCAN@tablatronix commented on GitHub (Nov 15, 2019):
it is legacy
@dontsovcmc commented on GitHub (Nov 15, 2019):
WiFiManager has bad architecture, any customization are hard to do. I want to see anybody who use this methods outside WiFiManager source code. It's difficult to refactor library if we have a lot of unused features.
@tablatronix commented on GitHub (Nov 15, 2019):
You keep saying "refactor" refator what?
What needs refactoring, I just refactored the entire thing to start organizing for future updates, v0 has no abstraction at all.
@dontsovcmc commented on GitHub (Nov 15, 2019):
I mean less code, less errors only =)