New settings / flags (#595)

[KipK]

FIxes #595
needs UI branch "newflags"

Not to be merged before UI migration

done:

• wizard_passed ( fw + ui )
• divert_type ( fw + ui )
• vehicle_data_src ( fw + ui )
• default_state (fw + ui )
• set AP ssid & password ( fw + ui )
• added /restart {"device": "gateway|evse"} to http api & mqtt + restartEvse() method in evseManager ( close Restarting controller via HTTP throws error 500 #479 ,

[KipK]

@jeremypoulter , I have segfault on divert_sim here. I have modified this (c3c68b7) in divert _sim. Any idea on what I've missed here ?

[KipK]

@jeremypoulter , when review is ok, don't merge it yet ( same for inputFilter too ). I'll have to merge the gui related branch to master first.

Can we decouple the daily builds from the gui commits now ?

[KipK]

ah also can you bump a new openevse lib version ?

[jeremypoulter]

Can we decouple the daily builds from the gui commits now ?

Just need to remove the workflow in the GUI repo

[jeremypoulter]

ah also can you bump a new openevse lib version ?

Did a new release this morning

[KipK]

Can we decouple the daily builds from the gui commits now ?

Just need to remove the workflow in the GUI repo

I've removed it and just put a gui build for testing.
This remind me to add some unit test on the gui later.

[jeremypoulter]

Yeah, testing would be excellent

[KipK]

Yeah, testing would be excellent

Cool thing we can do with svelte is testing each components easily.
https://sveltesociety.dev/recipes/testing-and-debugging/unit-testing-svelte-component

[KipK]

@mathieucarbou , I have changed here the mqtt /restart to /restart {payload} , with payload = "gateway" or "evse"

[mathieucarbou]

@mathieucarbou , I have changed here the mqtt /restart to /restart {payload} , with payload = "gateway" or "evse"

oh, nice :-)

[mathieucarbou]

@mathieucarbou , I have changed here the mqtt /restart to /restart {payload} , with payload = "gateway" or "evse"

FYI I've grabbed the restart changed in a branch and I have updated my automation when Zoé vehicle is connected.

I will report back this week-end hopefully is the sun is helping.

[KipK]

@KipK : i actually updated my automation that was doing a sequence of FD FE for Zoé to use that considering the rapi support will be removed. What other solution otherwise?

Still continue use RAPI until 5.0 release ?
You are posting issues on dev branches we haven't even merged yet, and things we're still working on, hold-on a bit ;)

[jeremypoulter]

Changing the pause state to disable should resolve this issue. However this removes the ability for the charger to detect if a vehicle is connected when paused.

@mathieucarbou You are doing the same thing as enabling that option so the warning applies weither you enable that option or not, but lets keep this discussion in #512

[jeremypoulter]

Need to also remove the replaced web files. IMHO I see little value in appending the file hash to the file name. The server should properly implement HTTP caching, so it should not be needed.

[KipK]

Need to also remove the replaced web files. IMHO I see little value in appending the file hash to the file name. The server should properly implement HTTP caching, so it should not be needed.

I think it's part of Svelte/Vite cache invalidation handler. Not sure it has consequencies to disable it, haven't tested yet.

[jeremypoulter]

Fine to leave it on I guess, but do then need to make sure all the headers are deleted before bringing in a new UI version. Really need to write a GitHub workflow to do that...

[KipK]

Really need to write a GitHub workflow to do that...

I have deleted the static files. Is there any need to have the generated statics of related UI commit ? Should stays empty like this on the git

[jeremypoulter]

We commit the static files so you don't have to build the UI to build the firmware.

[KipK]

We commit the static files so you don't have to build the UI to build the firmware.

ok, i'll update them here after the inputfilter merge.

[KipK]

UI changes merged in master, we're good on this one too.

[jeremypoulter]

Just the API doc errors to resolve

[jeremypoulter]

Oh ignore that, just noticed you added a change, let me check

[KipK]

Oh ignore that, just noticed you added a change, let me check

yes should be ok, I've installed the vscode extension and spotted the errors