mirror of
https://github.com/tzapu/WiFiManager.git
synced 2026-04-27 00:55:52 +03:00
[PR #487] [MERGED] make WIFI_MANAGER_MAX_PARAMS customizable #1630
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#1630
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?
📋 Pull Request Information
Original PR: https://github.com/tzapu/WiFiManager/pull/487
Author: @mtiutiu
Created: 1/22/2018
Status: ✅ Merged
Merged: 2/1/2018
Merged by: @tablatronix
Base:
master← Head:master📝 Commits (1)
8c906afmake WIFI_MANAGER_MAX_PARAMS customizable📊 Changes
1 file changed (+2 additions, -0 deletions)
View changed files
📝
WiFiManager.h(+2 -0)📄 Description
I think it's pretty useful to have this parameter customizable so that users can change it from outside if required (e.g platformio, make, etc).
For example when using platformio I can set this in platformio.ini:
build_flags = -DWIFI_MANAGER_MAX_PARAMS=20
This allows me to change the value at compile time without modifying the library source code. I also provided the default value for it when not defined outside or somewhere else - you can see that in the code changes that I want to propose.
This approach can be used for other defines in this library source code if desired/required.
Very nice work btw regarding this library - very useful. Thanks.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.