matijse / eufy-ha-mqtt-bridge

Receive Eufy alerts and thumbnails in Home Assistant via MQTT
124 stars 25 forks source link

Not saving config setting #63

Closed hamdanfadi closed 3 years ago

hamdanfadi commented 3 years ago

Hi, I have installed it, but when I try to save configuration. it doesn't allow me. Do I have to copy the content and copy it manually in custom-component folder?

potsu22 commented 3 years ago

same issue here, running addon via home assistant. Trying to edit the the config (via configuration tab of addon) when you hit save it just reverts to defaults.

attempting to run the addon without changing config results in error Missing option 'home_assistant' in root in Eufy Home Assistant MQTT Bridge (f1c878cb_eufy-ha-mqtt-bridge).

hamdanfadi commented 3 years ago

Yes it seems last commit that he made broke it. It was working fine 3 days ago

On 21/04/2021, at 13:25, potsu22 @.***> wrote:



same issue here, running addon via home assistant. Trying to edit the the config (via configuration tab of addon) when you hit save it just reverts to defaults.

attempting to run the addon without changing config results in error Missing option 'home_assistant' in root in Eufy Home Assistant MQTT Bridge (f1c878cb_eufy-ha-mqtt-bridge).

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmatijse%2Feufy-ha-mqtt-bridge%2Fissues%2F63%23issuecomment-823707682&data=04%7C01%7C%7C94ac6ea2b48b4d4d2d4908d904645a1f%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637545651315980453%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=DnAhmbJ4H9%2FHIjteRPBKmOftTzWXYA9jhsyxVBUHptA%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALINWF55NNGBZPAKZVGY5XLTJYSQTANCNFSM43JIPWEA&data=04%7C01%7C%7C94ac6ea2b48b4d4d2d4908d904645a1f%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637545651316030424%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=XDiChZ4vX4T96B0CWPS6w9DbyoouW80MQS5YyAfTTaQ%3D&reserved=0.

MaxWinterstein commented 3 years ago

Should be fixed with newest add-on release v1.16.1. See #62

ajvdw commented 3 years ago

same here on 1.16.0 . After updating to the latest version I got: "attempting to run the addon without changing config results in error Missing option 'home_assistant' in root in Eufy Home Assistant MQTT Bridge (f1c878cb_eufy-ha-mqtt-bridge)."

Reinstalled, I attempted to change the default config. It didn't allow me to. I can't start the add-on anymore.....

MaxWinterstein commented 3 years ago

same here. After updating to the latest version I got: "attempting to run the addon without changing config results in error Missing option 'home_assistant' in root in Eufy Home Assistant MQTT Bridge (f1c878cb_eufy-ha-mqtt-bridge)."

Reinstalled, I attempted to change the default config. It didn't allow me to. I can't start the add-on anymore.....

Did you install v1.16.1? Is was released just a few minutes ago. You might have to relead the addon-store manually image

hamdanfadi commented 3 years ago

Let’s wait for a fix hopefully soon

On 21/04/2021, at 19:28, Max Winterstein @.***> wrote:



same here. After updating to the latest version I got: "attempting to run the addon without changing config results in error Missing option 'home_assistant' in root in Eufy Home Assistant MQTT Bridge (f1c878cb_eufy-ha-mqtt-bridge)."

Reinstalled, I attempted to change the default config. It didn't allow me to. I can't start the add-on anymore.....

Did you install v1.16.1? Is was released just a few minutes ago. You might have to relead the addon-store manually [image]https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F5927148%2F115513920-e0bf9700-a283-11eb-9bf8-8cbad4c4cf1b.png&data=04%7C01%7C%7C911fb93351874b8946d908d904970eb4%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637545869089344869%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=a%2FLxBexkLyEzAYCF41DglHHkRARUyK6HYbDW2O7NTzk%3D&reserved=0

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmatijse%2Feufy-ha-mqtt-bridge%2Fissues%2F63%23issuecomment-823842650&data=04%7C01%7C%7C911fb93351874b8946d908d904970eb4%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637545869089344869%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=PPzrVSM%2BfIkcSg2hq71mRUGvOrLmQnTFN%2BYx4zYXs%2Fc%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALINWF64SH4QESU2AC7G5H3TJZ5BXANCNFSM43JIPWEA&data=04%7C01%7C%7C911fb93351874b8946d908d904970eb4%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637545869089354861%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=yfMzMeU4c5hX0Lm7u%2FStBBmnYks3kVNLcbRI%2B%2FX8g8A%3D&reserved=0.

ajvdw commented 3 years ago

same here. After updating to the latest version I got: "attempting to run the addon without changing config results in error Missing option 'home_assistant' in root in Eufy Home Assistant MQTT Bridge (f1c878cb_eufy-ha-mqtt-bridge)." Reinstalled, I attempted to change the default config. It didn't allow me to. I can't start the add-on anymore.....

Did you install v1.16.1? Is was released just a few minutes ago. You might have to relead the addon-store manually image

works again on 1.16.1 after adding the two lines below to my config.

home_assistant:
   off_delay: 5
MaxWinterstein commented 3 years ago

same here. After updating to the latest version I got: "attempting to run the addon without changing config results in error Missing option 'home_assistant' in root in Eufy Home Assistant MQTT Bridge (f1c878cb_eufy-ha-mqtt-bridge)." Reinstalled, I attempted to change the default config. It didn't allow me to. I can't start the add-on anymore.....

Did you install v1.16.1? Is was released just a few minutes ago. You might have to relead the addon-store manually image

works again on 1.16.1 after adding the two lines below to my config.

home_assistant:
   off_delay: 5

Is is possible that you overwrite the whole config via copy/paste? This sets it back to default for me too.

hamdanfadi commented 3 years ago

New version fixed the issue my dear

From: Max Winterstein @.> Sent: Wednesday, 21 April 2021 7:48 PM To: matijse/eufy-ha-mqtt-bridge @.> Cc: Fadi Hamdan @.>; Author @.> Subject: Re: [matijse/eufy-ha-mqtt-bridge] Not saving config setting (#63)

same here. After updating to the latest version I got: "attempting to run the addon without changing config results in error Missing option 'home_assistant' in root in Eufy Home Assistant MQTT Bridge (f1c878cb_eufy-ha-mqtt-bridge)." Reinstalled, I attempted to change the default config. It didn't allow me to. I can't start the add-on anymore.....

Did you install v1.16.1? Is was released just a few minutes ago. You might have to relead the addon-store manually [image]https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F5927148%2F115513920-e0bf9700-a283-11eb-9bf8-8cbad4c4cf1b.png&data=04%7C01%7C%7C5fa3641865ec4807f20208d90499d24e%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637545880960523515%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=E7j3uQZBNd7KHv3j6RYRwtNnVyI92YCj8ctlE6hIn8M%3D&reserved=0

works again on 1.16.1 after adding the two lines below to my config.

home_assistant:

off_delay: 5

Is is possible that you overwrite the whole config via copy/paste? This sets it back to default for me too.

- You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmatijse%2Feufy-ha-mqtt-bridge%2Fissues%2F63%23issuecomment-823854695&data=04%7C01%7C%7C5fa3641865ec4807f20208d90499d24e%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637545880960533505%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=qbp%2F4RTZU%2F%2BtWkmPrDMR1xb1V0YzDgi3YUpgBVXZTj0%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALINWFYH5RYB3DB4T6NZ7NTTJZ7L5ANCNFSM43JIPWEA&data=04%7C01%7C%7C5fa3641865ec4807f20208d90499d24e%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637545880960543497%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=qt%2BdK0g%2FhAzyxfso1weXt%2BXDdCCl7tzawbHX%2BGbon2U%3D&reserved=0.

hamdanfadi commented 3 years ago

The new version it fixed the issue. Thanks We are waiting for the live stream function. I would suggest to merge it with nonsleepr / ha-eufy-security

https://github.com/nonsleepr/ha-eufy-security.git

hamdanfadi commented 3 years ago

Also, it will be nice to add feature to change guard mode (armed, disarmed ....etc.

Let all support @matijse please let us know your givemecopfee account.

matijse commented 3 years ago

Merge with nonsleepr integration is not possible as that is written in Python. Guard mode and live stream will be implemented after the rewrite...

hamdanfadi commented 3 years ago

Thank mastijse, What you mean by rewrie?

Cheers, From: matijse @.> Sent: Thursday, 22 April 2021 7:08 AM To: matijse/eufy-ha-mqtt-bridge @.> Cc: Fadi Hamdan @.>; Author @.> Subject: Re: [matijse/eufy-ha-mqtt-bridge] Not saving config setting (#63)

Merge with nonsleepr integration is not possible as that is written in Python. Guard mode and live stream will be implemented after the rewrite...

- You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmatijse%2Feufy-ha-mqtt-bridge%2Fissues%2F63%23issuecomment-824291757&data=04%7C01%7C%7Cebd5e142e3154c9e50ff08d904f8d67b%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637546289050912861%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=cM7Qnq0LbKZZgA%2B%2BL%2Ffz3Q01HPaV%2BPMF9NUMqFk2JOI%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALINWF3VFENHNROWK2VU3LLTJ4PCPANCNFSM43JIPWEA&data=04%7C01%7C%7Cebd5e142e3154c9e50ff08d904f8d67b%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637546289050922854%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=QF8fTaXbIMRIlMz7Amwml5l%2BOaXWDxiD4vj%2BJcz0gcg%3D&reserved=0.

matijse commented 3 years ago

I am changing the code for this script to use a new library (https://github.com/bropat/eufy-security-client) with a lot more features, but that needs a rewrite of the code (which takes time)