Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

After a container restart Alerts, Web Server lose their config #31

Closed
mihsu81 opened this issue Sep 10, 2021 · 4 comments
Closed

After a container restart Alerts, Web Server lose their config #31

mihsu81 opened this issue Sep 10, 2021 · 4 comments

Comments

@mihsu81
Copy link
Contributor

mihsu81 commented Sep 10, 2021

Hi @basnijholt,

After a restart of the container the Alerts and Web Server tabs lose their config.

Also the settings added in the ThingSpeak tab are lost after navigating away from the tab.

Thank you in advance for your help in getting this solved.
Mihai

@fliphess
Copy link
Collaborator

@mihsu81

I'm afraid you are correct: On container start some updates on the configuration are done using a startup script, but the config itself is very stateless and as such on container restart the manually configured settings are gone.

I'd love to make some edits to change this, but I'm currently a bit busy with other things.

I'll keep this issue open so if I can find some time to change this, I can have a look! :)

@mihsu81
Copy link
Contributor Author

mihsu81 commented Nov 23, 2021

Hi @fliphess,
I've made some adjustments in a fork of your repo to include E-mail and ThingSpeak settings and added 3 more MQTT settings (All Messages, QOS and Retransmit). If you'd like I can do a PR.
Thanks.

@fliphess
Copy link
Collaborator

Hey @mihsu81

I'd love to add it!

Feel free to create a PR so we can merge it in over here :)

mihsu81 added a commit to mihsu81/addon-otmonitor that referenced this issue Nov 24, 2021
Added Email and ThingSpeak settings and 3 more MQTT options (All Messages, QOS and Retransmit)
basnijholt#31
fliphess pushed a commit that referenced this issue Jan 8, 2022
Added Email and ThingSpeak settings and 3 more MQTT options (All Messages, QOS and Retransmit)
#31
@fliphess
Copy link
Collaborator

fliphess commented Jan 8, 2022

I've merged your PR, I think we can close this :)

If you think we should reopen it, please let me know :) Thanks a lot! 👍 🥇

@fliphess fliphess closed this as completed Jan 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants