[GH-ISSUE #968] dublicate function: /0wifi and /wifi&refresh #822

Closed
opened 2026-02-28 01:27:12 +03:00 by kerem · 4 comments
Owner

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_WIFI

no scan will be /wifi without argument:
"<form action='/wifi' method='get'><button>Configure WiFi (No Scan)</button></form><br/>\n", // MENU_WIFINOSCAN

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_WIFI` no scan will be /wifi without argument: `"<form action='/wifi' method='get'><button>Configure WiFi (No Scan)</button></form><br/>\n", // MENU_WIFINOSCAN`
kerem closed this issue 2026-02-28 01:27:12 +03:00
Author
Owner

@tablatronix commented on GitHub (Nov 15, 2019):

it is legacy

<!-- gh-comment-id:554437645 --> @tablatronix commented on GitHub (Nov 15, 2019): it is legacy
Author
Owner

@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.

<!-- gh-comment-id:554447279 --> @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.
Author
Owner

@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.

<!-- gh-comment-id:554565225 --> @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.
Author
Owner

@dontsovcmc commented on GitHub (Nov 15, 2019):

I mean less code, less errors only =)

<!-- gh-comment-id:554571056 --> @dontsovcmc commented on GitHub (Nov 15, 2019): I mean less code, less errors only =)
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/WiFiManager#822
No description provided.