[GH-ISSUE #767] Testing ota branch #641

Closed
opened 2026-02-28 01:26:21 +03:00 by kerem · 16 comments
Owner

Originally created by @martinberlin on GitHub (Nov 17, 2018).
Original GitHub issue: https://github.com/tzapu/WiFiManager/issues/767

So far is a bit confusing.

Scenario
Updated Wifi Manager to ota branch
/Arduino/libraries/WiFiManager$ git status
On branch ota
Your branch is up to date with 'origin/ota'

In my sketch : Added the "update" option to menu vector.

What is expected to happen
That update option appears when WiFi manager captive portal starts

What is happening
options
A new Erase option appears (But I didn't add erase to the menu array)
Note: My comment on the image is wrong, Erase option appears automatically, has nothing to do with adding the additional "update" option to the menu

No update option appears. If I go to info, I see there is a /u Route, clicking on it I get:
File Not Found

URI: /u
Method: GET
Arguments: 0

Can you please explain in a short note how to test this ?
Thanks in advance

UPDATE: Now I see this github.com/tzapu/WiFiManager@0ec7a58b7b
I will try again later and update this. But still don't get fully how it should work.
Maybe you can provide a working demo in your /examples Folder ?

Originally created by @martinberlin on GitHub (Nov 17, 2018). Original GitHub issue: https://github.com/tzapu/WiFiManager/issues/767 So far is a bit confusing. Scenario Updated Wifi Manager to ota branch /Arduino/libraries/WiFiManager$ git status On branch ota Your branch is up to date with 'origin/ota' In my sketch : Added the "update" option to menu vector. **What is expected to happen** That update option appears when WiFi manager captive portal starts **What is happening** ![options](https://user-images.githubusercontent.com/2692928/48657565-fdac6c00-ea32-11e8-9646-09f5b3e5f5a4.png) A new Erase option appears (But I didn't add erase to the menu array) Note: My comment on the image is wrong, Erase option appears automatically, has nothing to do with adding the additional "update" option to the menu No update option appears. If I go to info, I see there is a /u Route, clicking on it I get: File Not Found URI: /u Method: GET Arguments: 0 Can you please explain in a short note how to test this ? Thanks in advance UPDATE: Now I see this https://github.com/tzapu/WiFiManager/commit/0ec7a58b7b0888a1ef261b38694f7a9e5633773e I will try again later and update this. But still don't get fully how it should work. Maybe you can provide a working demo in your /examples Folder ?
kerem 2026-02-28 01:26:21 +03:00
  • closed this issue
  • added the
    bug
    Branch
    labels
Author
Owner

@tablatronix commented on GitHub (Nov 17, 2018):

what is your menu code?

<!-- gh-comment-id:439615667 --> @tablatronix commented on GitHub (Nov 17, 2018): what is your menu code?
Author
Owner

@martinberlin commented on GitHub (Nov 17, 2018):

Is the same as you can see in the develop branch of the FS2 project. I copied it from your example if I remember well.

<!-- gh-comment-id:439622499 --> @martinberlin commented on GitHub (Nov 17, 2018): Is the same as you can see in the develop branch of the FS2 project. I copied it from your example if I remember well.
Author
Owner

@tablatronix commented on GitHub (Nov 17, 2018):

I dont see any menu code in develop, did you push it?

<!-- gh-comment-id:439627074 --> @tablatronix commented on GitHub (Nov 17, 2018): I dont see any menu code in develop, did you push it?
Author
Owner

@martinberlin commented on GitHub (Nov 18, 2018):

Good morning Shawn,
Just added a feature/ota branch and added "update" in the menu:

github.com/martinberlin/FS2@a8bf7ec6d7

Note: I'm using the latest updates in your ota branch. There if you add "sep" to the menu

std::vector<const char *> menu = {"wifi", "param", "info", "update", "sep"};
wm.setMenu(menu);

It will make appear automatically the Erase red button. But update for me is completely ignored, there is no update coming as an option.

<!-- gh-comment-id:439676517 --> @martinberlin commented on GitHub (Nov 18, 2018): Good morning Shawn, Just added a feature/ota branch and added "update" in the menu: https://github.com/martinberlin/FS2/commit/a8bf7ec6d7a8993c588d38e57efe0b51116fe514 Note: I'm using the latest updates in your ota branch. There if you add "sep" to the menu std::vector<const char *> menu = {"wifi", "param", "info", "update", "sep"}; wm.setMenu(menu); It will make appear automatically the Erase red button. But **update** for me is completely ignored, there is no update coming as an option.
Author
Owner

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

sorry about that, the menu token for update was missing and you found a bug, fixing

<!-- gh-comment-id:439705503 --> @tablatronix commented on GitHub (Nov 18, 2018): sorry about that, the menu token for update was missing and you found a bug, fixing
Author
Owner

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

Fixed thanks!

<!-- gh-comment-id:439705888 --> @tablatronix commented on GitHub (Nov 18, 2018): Fixed thanks!
Author
Owner

@martinberlin commented on GitHub (Nov 19, 2018):

Works fantastic, thanks!!!

<!-- gh-comment-id:440006568 --> @martinberlin commented on GitHub (Nov 19, 2018): Works fantastic, thanks!!!
Author
Owner

@rodri16 commented on GitHub (Nov 27, 2019):

Hi, I am also testing OTA branch and when trying to upload a code I get an error, it says

*WM: [3] -> 192.168.4.1 
Update: Blink.ino
sleep disable
*WM: [3] -> 192.168.4.1 
.*WM: [3] -> 192.168.4.1 
ERROR[10]: Magic byte is wrong, not 0xE9
*WM: [2] <- Handle update don
<!-- gh-comment-id:559231211 --> @rodri16 commented on GitHub (Nov 27, 2019): Hi, I am also testing OTA branch and when trying to upload a code I get an error, it says ``` *WM: [3] -> 192.168.4.1 Update: Blink.ino sleep disable *WM: [3] -> 192.168.4.1 .*WM: [3] -> 192.168.4.1 ERROR[10]: Magic byte is wrong, not 0xE9 *WM: [2] <- Handle update don ```
Author
Owner

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

Ill take a look and merge in updates

<!-- gh-comment-id:559239948 --> @tablatronix commented on GitHub (Nov 27, 2019): Ill take a look and merge in updates
Author
Owner

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

Did you check esp lib for this error?

<!-- gh-comment-id:559240413 --> @tablatronix commented on GitHub (Nov 27, 2019): Did you check esp lib for this error?
Author
Owner

@rodri16 commented on GitHub (Nov 27, 2019):

I made it work sorry, it was a wrong file!!! I was trying tih .ino file and I have to use .bin

 *WM: [3] -> 192.168.4.1 
Update: Blink.ino.d1_mini.bin
sleep disable
*WM: [3] -> 192.168.4.1 
.*WM: [3] -> 192.168.4.1 
Updated: 261824 bytes
Rebooting...
*WM: [2] <- Handle update done 
*WM: [3] -> 192.168.4.1 
*WM: [1] [OTA] update ok 
del if0
station: 9e:f2:09:a1:3b:5a leave, AID = 1
rm 1
bcn 0
del if1
usl

Is OTA branch updated with last DEV changes?
Thanks @tablatronix

<!-- gh-comment-id:559259530 --> @rodri16 commented on GitHub (Nov 27, 2019): I made it work sorry, it was a wrong file!!! I was trying tih .ino file and I have to use .bin ``` *WM: [3] -> 192.168.4.1 Update: Blink.ino.d1_mini.bin sleep disable *WM: [3] -> 192.168.4.1 .*WM: [3] -> 192.168.4.1 Updated: 261824 bytes Rebooting... *WM: [2] <- Handle update done *WM: [3] -> 192.168.4.1 *WM: [1] [OTA] update ok del if0 station: 9e:f2:09:a1:3b:5a leave, AID = 1 rm 1 bcn 0 del if1 usl ``` Is OTA branch updated with last DEV changes? Thanks @tablatronix
Author
Owner

@rodri16 commented on GitHub (Dec 12, 2019):

Hi @tablatronix
I moved to dev branch but left "update" on the menu. Adding

#define USEOTA
// enable OTA
#ifdef USEOTA
#include <WiFiUdp.h>
#include <ArduinoOTA.h>
#endif

But update button doesn't apper in WM menu

<!-- gh-comment-id:565069640 --> @rodri16 commented on GitHub (Dec 12, 2019): Hi @tablatronix I moved to dev branch but left "update" on the menu. Adding ``` #define USEOTA // enable OTA #ifdef USEOTA #include <WiFiUdp.h> #include <ArduinoOTA.h> #endif ``` But update button doesn't apper in WM menu
Author
Owner

@tablatronix commented on GitHub (Dec 12, 2019):

I gotta update that branch

<!-- gh-comment-id:565131801 --> @tablatronix commented on GitHub (Dec 12, 2019): I gotta update that branch
Author
Owner

@tablatronix commented on GitHub (Dec 12, 2019):

merged , let me test it real quick

<!-- gh-comment-id:565133579 --> @tablatronix commented on GitHub (Dec 12, 2019): merged , let me test it real quick
Author
Owner

@tablatronix commented on GitHub (Dec 12, 2019):

OTA is in the OTA branch, not dev

<!-- gh-comment-id:565134549 --> @tablatronix commented on GitHub (Dec 12, 2019): OTA is in the OTA branch, not dev
Author
Owner

@tablatronix commented on GitHub (Dec 12, 2019):

#981
realized there was no issue for this

<!-- gh-comment-id:565137177 --> @tablatronix commented on GitHub (Dec 12, 2019): #981 realized there was no issue for this
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#641
No description provided.