[GH-ISSUE #659] setConfigPortalBlocking and additional parameter portail not compatible #550

Open
opened 2026-02-28 01:25:50 +03:00 by kerem · 14 comments
Owner

Originally created by @spilz87 on GitHub (Jul 15, 2018).
Original GitHub issue: https://github.com/tzapu/WiFiManager/issues/659

I'm not able to use setConfigPortalBlocking(false) and add parameters on portail on the same project, I tried to merge the 2 exemples and each time I got a reset loop

Some one as succed on it or is it an issue ?

Originally created by @spilz87 on GitHub (Jul 15, 2018). Original GitHub issue: https://github.com/tzapu/WiFiManager/issues/659 I'm not able to use setConfigPortalBlocking(false) and add parameters on portail on the same project, I tried to merge the 2 exemples and each time I got a reset loop Some one as succed on it or is it an issue ?
Author
Owner

@tablatronix commented on GitHub (Jul 18, 2018):

is this still an issue?

<!-- gh-comment-id:406058747 --> @tablatronix commented on GitHub (Jul 18, 2018): is this still an issue?
Author
Owner

@spilz87 commented on GitHub (Jul 18, 2018):

For me yes it is, I'm not able to merge the both exemples

<!-- gh-comment-id:406068659 --> @spilz87 commented on GitHub (Jul 18, 2018): For me yes it is, I'm not able to merge the both exemples
Author
Owner

@tablatronix commented on GitHub (Jul 18, 2018):

do you have the latest commits?
Did you place your param object global or make it static?

<!-- gh-comment-id:406071151 --> @tablatronix commented on GitHub (Jul 18, 2018): do you have the latest commits? Did you place your param object global or make it static?
Author
Owner

@spilz87 commented on GitHub (Jul 18, 2018):

I download the last development branch last week end

<!-- gh-comment-id:406075876 --> @spilz87 commented on GitHub (Jul 18, 2018): I download the last development branch last week end
Author
Owner

@tablatronix commented on GitHub (Jul 18, 2018):

It has been updated since then

<!-- gh-comment-id:406076784 --> @tablatronix commented on GitHub (Jul 18, 2018): It has been updated since then
Author
Owner

@spilz87 commented on GitHub (Jul 18, 2018):

Ok sorry
I will download the new one and try it again

<!-- gh-comment-id:406077297 --> @spilz87 commented on GitHub (Jul 18, 2018): Ok sorry I will download the new one and try it again
Author
Owner

@spilz87 commented on GitHub (Jul 18, 2018):

I will let you know

<!-- gh-comment-id:406077335 --> @spilz87 commented on GitHub (Jul 18, 2018): I will let you know
Author
Owner

@irwanayeah commented on GitHub (Aug 29, 2018):

hello @ spilz87 @tablatronix , is there a solution? because I got the same thing

I use the sample WifiManager \ examples \ DEV \ OnDemandConfigPortal \ OnDemandConfigPortal.ino

then add lines
setConfigPortalBlocking (false); on setup
process (); on the loop

parameters never seen.

<!-- gh-comment-id:417079899 --> @irwanayeah commented on GitHub (Aug 29, 2018): hello @ spilz87 @tablatronix , is there a solution? because I got the same thing I use the sample WifiManager \ examples \ DEV \ OnDemandConfigPortal \ OnDemandConfigPortal.ino then add lines setConfigPortalBlocking (false); on setup process (); on the loop parameters never seen.
Author
Owner

@tablatronix commented on GitHub (Aug 29, 2018):

Your parameters are setup in setup then they are out of scope in loop. You have to put then somewhere else or make them static or something

<!-- gh-comment-id:417096552 --> @tablatronix commented on GitHub (Aug 29, 2018): Your parameters are setup in setup then they are out of scope in loop. You have to put then somewhere else or make them static or something
Author
Owner

@irwanayeah commented on GitHub (Aug 29, 2018):

work perfect. thank @tablatronix

<!-- gh-comment-id:417098809 --> @irwanayeah commented on GitHub (Aug 29, 2018): work perfect. thank @tablatronix
Author
Owner

@tablatronix commented on GitHub (Aug 29, 2018):

Yeah that is one of the problems with the non blocking and has to be documented and a new example made for it, unless we can make it better, like store parm objects in wm.
Then there is the issue of cleaning up when its global etc.

Ideas welcome

<!-- gh-comment-id:417103632 --> @tablatronix commented on GitHub (Aug 29, 2018): Yeah that is one of the problems with the non blocking and has to be documented and a new example made for it, unless we can make it better, like store parm objects in wm. Then there is the issue of cleaning up when its global etc. Ideas welcome
Author
Owner

@irwanayeah commented on GitHub (Aug 29, 2018):

hi @tablatronix , how about function setSaveConfig Callback(save Config Callback); never fires after saving value of parameters.

<!-- gh-comment-id:417139846 --> @irwanayeah commented on GitHub (Aug 29, 2018): hi @tablatronix , how about function setSaveConfig Callback(save Config Callback); never fires after saving value of parameters.
Author
Owner

@tablatronix commented on GitHub (Aug 29, 2018):

Yeah I keep forgetting to work on that. Can you create an issue so I dont forget again.

Since params can now be saved seperate from wifi I need to add new callbacks also

<!-- gh-comment-id:417145373 --> @tablatronix commented on GitHub (Aug 29, 2018): Yeah I keep forgetting to work on that. Can you create an issue so I dont forget again. Since params can now be saved seperate from wifi I need to add new callbacks also
Author
Owner

@irwanayeah commented on GitHub (Aug 29, 2018):

ok

<!-- gh-comment-id:417145629 --> @irwanayeah commented on GitHub (Aug 29, 2018): ok
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#550
No description provided.