WiFiManager icon indicating copy to clipboard operation
WiFiManager copied to clipboard

Update FS examples to use Arduinojson 6 (Fixes #1059)

Open alfo opened this issue 5 years ago • 12 comments

Small direct updates to change the AutoConnectWithFSParameters and AutoConnectWithFSParametersAndCustomIP examples to make them compatible with Arduinojson 6

Fixes #1059

alfo avatar May 22 '20 11:05 alfo

I thought this was already done..

tablatronix avatar May 22 '20 14:05 tablatronix

There is an open PR but it doesn't cover both example files (and seems to change a lot of unrelated code too...)

alfo avatar May 22 '20 14:05 alfo

It seems to be fixed in hotfixes, not sure if I ever merged into master.

Either way it needs to be fixed in development branch also, not sure if I will be able to merge them without conflicts , ill check also

Let me look into the branches and see whats up, maybe that pr just missed some, ill try to clean them up today, thanks!

tablatronix avatar May 22 '20 14:05 tablatronix

No worries! I left the development branch examples alone because it seems like they're not finalised yet

alfo avatar May 22 '20 14:05 alfo

Yup it looks like I never made a release for the hotfixes changes, shoot.

Sorry about that, ill double check which one is better and merge them, thanks

tablatronix avatar May 22 '20 15:05 tablatronix

I changed your target to hotfixes, here is the conflicts with what was already committed, I asked that that one be backward compatible, let me know if something is missing, we can probably just skip your PR, sorry to waste your time. I appreciate it.

https://github.com/tzapu/WiFiManager/pull/1060/conflicts

tablatronix avatar May 22 '20 15:05 tablatronix

Yeah, that previous PR looks fine to me. The only thing is whether we use Static or Dynamic JSON documents, and what size they are, but ultimately this is example code so it's up to the end user.

alfo avatar May 25 '20 10:05 alfo

Resolved conflicts and slightly tidied up

alfo avatar May 25 '20 12:05 alfo

Your going to jail for fraud

Falludh avatar Mar 10 '24 14:03 Falludh

lol, what we miss?

tablatronix avatar Mar 10 '24 20:03 tablatronix

Don’t worry Shawn, we’ll put money in your commissary account if you go to the GitHub slammer.

chriskinal avatar Mar 10 '24 21:03 chriskinal

Don’t worry Shawn, we’ll put money in your commissary account if you go to the GitHub slammer.

Dont worry i know what Your doing and your going to jail

Falludh avatar Mar 19 '24 20:03 Falludh