serkri / SmartEVSE-3

Smart Electric Vehicle Charging Station (EVSE)
MIT License
72 stars 27 forks source link

Some MQTT setting fields are empty after page reload #166

Closed hmmbob closed 1 year ago

hmmbob commented 1 year ago

I've succesfully connected the SmartEVSE to my mqtt server (great!), but when I now reopen the mqtt settings tab some fields are empty:

image

I expected to see the current config here (maybe without the password or so).

dingo35 commented 1 year ago

I would expect that too. But since it is hardly worth the effort (how often do you change your mqtt config?) I didnt bother the original author.

Feel free to post a Pull Request!

hmmbob commented 1 year ago

Is there any way to connect to the SmartEVSE and edit the index.html directly, or is there always a new build needed?

I have no experience with the latter (I always update with your provided binaries), so I haven't tested this locally. However, given that the key names were just not matching, I'm 99% sure this is fixed with the PR submitted.

edit: got to learn about "overrides" in Chrome developer tools: the changes fix the issue locally.

dingo35 commented 1 year ago

You can test your local index.html easy by editing the line that has 10.0.0.36 in it; comment the line above it, change it to your own smartevse's ip address, and if you approach file:///path to your local index.html you can see immmediate results

hmmbob commented 1 year ago

The "local override" (see edit above) worked - it's confirmed tested

dingo35 commented 1 year ago

Yup we were typing in parallel; hope to test this afternoon.

dingo35 commented 1 year ago

@hmmbob Thanks for your work!

deekoowee commented 1 year ago

@hmmbob , how did you reach the mqtt config page ? image My Smart EVSE main page looks like this; image2

hmmbob commented 1 year ago

Did you flash spiffs too? Or only firmware.bin?

deekoowee commented 1 year ago

I have flashed only firmware.bin Now I flashed spiffs too and config functionality appears on main screen. Thanks for helping on this.