LAB02-Research / HASS.Agent-Integration

HASS.Agent's Home Assistant integration. Adds notifications and mediaplayer capabilities to HASS.Agent - a Windows based client (companion app) for Home Assistant.
MIT License
101 stars 17 forks source link

HASS.Agent no longer sends notifications after updating to Home Assistant Core 2024.5.0 #47

Open peterskajr opened 2 months ago

peterskajr commented 2 months ago

After updating Home Assistant to Home Assistant Core 2024.5.0, lost the ability to send notifications back to the Windows PC. The Test Notification button in the HASS Configuration page still sent the test notification, but any automations that were previously written to send notifications no longer worked. The pc no longer showed up as a service: notifiy.windowspc option. Reverting back my Home Assistant to the previous version of 2024.4.4 restored the HASS.Agent notifications.

rrozema commented 2 months ago

Can it be related to https://developers.home-assistant.io/blog/2024/04/10/new-notify-entity-platform/?

Protoncek commented 2 months ago

Also media player doesn't work anymore - it's unavailable (in all 3 of my instances). EDIT: i just went back to 2024.4.x, all works again. So problem is indeed 2024.5 HA

atax112 commented 2 months ago

Same, after the update got a bunch of repairs, hass.agent notification not available anymore...

WizGnome commented 2 months ago

Same for me, Media Player unavailable after updating to 2024.5

Protoncek commented 2 months ago

Since author of this addon doesn't actively work on this (latest version is 2022) i think that probably doesn't read these issues also. So, i found a solution on a more recent fork of hass agent: https://github.com/hass-agent/HASS.Agent-Integration/issues/2 (corrected link) After this manual modification all works again.

atax112 commented 2 months ago

Since author of this addon doesn't actively work on this (latest version is 2022) i think that probably doesn't read these issues also. So, i found a solution on a more recent fork of hass agent: https://github.com/hass-agent/HASS.Agent-Integration/issues/2 After this manual modification all works again.

Hey, I guess the link is wrong? Doesnt point to what you mean... Edit: the text was ok, the link not...

https://github.com/hass-agent/HASS.Agent-Integration/issues/2

Ok, so, quick&dirty fix for anyone that can/wants edit the files manually :D

Navigate to "/config/custom_components/hass_agent" - either inside the container or via some editor when using the supervisor enabled HA. Edit "init.py" file (for example "vi init.py") Add "async" in following places: a) line 27: "async def update_device_info(hass: HomeAssistant, entry: ConfigEntry, new_device_info):" b) line 139: "async def updated(message: ReceiveMessage):" Add "await" in following places: a) line 144: "await update_device_info(hass, entry, payload)" Restart Home Assistant What I did was do the above changes on version lower than 2024.5.0, confirmed that notifications and media player are working, then I upgraded to 2024.5.0 and tested the same - appears to work as expected.

image

atax112 commented 2 months ago

Yeah, figured it out in the meantime, updated my comment with the linked potential fix. Many thanks @Protoncek !

Protoncek commented 2 months ago

Thanks for correction...

atax112 commented 2 months ago

Did the steps above, I can confirm working!

v1k70rk4 commented 2 months ago

Thanks to everyone! I'm very sorry that this plugin is not progressing further :( I'm afraid of what will happen if a more serious problem comes.

zanix commented 2 months ago

Thanks to everyone! I'm very sorry that this plugin is not progressing further :( I'm afraid of what will happen if a more serious problem comes.

Seems like it will be continued here https://github.com/hass-agent

Axsgranted commented 2 months ago

Confirmed fix suggested by atax112 works.

Failure404 commented 2 months ago

I think line 124 also needs an 'await'

    await update_device_info(hass, entry, response_json)
jonny190 commented 2 months ago

Confirmed fix suggested by atax112 works.

CT2HUU commented 2 months ago

Since author of this addon doesn't actively work on this (latest version is 2022) i think that probably doesn't read these issues also. So, i found a solution on a more recent fork of hass agent: https://github.com/hass-agent/HASS.Agent-Integration/issues/2 After this manual modification all works again.

Hey, I guess the link is wrong? Doesnt point to what you mean... Edit: the text was ok, the link not...

hass-agent/HASS.Agent-Integration#2 (comment)

Ok, so, quick&dirty fix for anyone that can/wants edit the files manually :D

Navigate to "/config/custom_components/hass_agent" - either inside the container or via some editor when using the supervisor enabled HA. Edit "init.py" file (for example "vi init.py") Add "async" in following places: a) line 27: "async def update_device_info(hass: HomeAssistant, entry: ConfigEntry, new_device_info):" b) line 139: "async def updated(message: ReceiveMessage):" Add "await" in following places: a) line 144: "await update_device_info(hass, entry, payload)" Restart Home Assistant What I did was do the above changes on version lower than 2024.5.0, confirmed that notifications and media player are working, then I upgraded to 2024.5.0 and tested the same - appears to work as expected.

image

Very dangerous for someone like me without any background on coding... I think I will have to wait for a fix in Agent :(

Protoncek commented 2 months ago

No particular danger here: open studio code server (if you don't have it install it from addons), find file "init.py" in folder, specified above, then

Worst thing it can happen is that hass.agent still won't work, but HA will - in this case:

But, if you'd like to go back to original:

CT2HUU commented 2 months ago

No particular danger here: open studio code server (if you don't have it install it from addons), find file "init.py" in folder, specified above, then

  • right-click on it and click CTRL+C, followed by CTRL+V. A new file with "__ copy.py" will appear. That way you made a copy of original.
  • made above changes and restart HA

Worst thing it can happen is that hass.agent still won't work, but HA will - in this case:

  • see what you did wrong, correct and restart again.

But, if you'd like to go back to original:

  • delete "init.py" you just changed
  • click on copy you made eariler
  • press F2 and delete part "_copy" at the end of file. Make sure that you don't delete .py
  • restart HA again and you're back on original Since without these changes thing doesn't work you have nothing to loose trying, right?

ok, I will give it a try. thanks for the step by step procedure.

CT2HUU commented 2 months ago

Since author of this addon doesn't actively work on this (latest version is 2022) i think that probably doesn't read these issues also. So, i found a solution on a more recent fork of hass agent: https://github.com/hass-agent/HASS.Agent-Integration/issues/2 After this manual modification all works again.

Hey, I guess the link is wrong? Doesnt point to what you mean... Edit: the text was ok, the link not...

hass-agent#2

Ok, so, quick&dirty fix for anyone that can/wants edit the files manually :D

Navigate to "/config/custom_components/hass_agent" - either inside the container or via some editor when using the supervisor enabled HA. Edit "init.py" file (for example "vi init.py") Add "async" in following places: a) line 27: "async def update_device_info(hass: HomeAssistant, entry: ConfigEntry, new_device_info):" b) line 139: "async def updated(message: ReceiveMessage):" Add "await" in following places: a) line 144: "await update_device_info(hass, entry, payload)" Restart Home Assistant What I did was do the above changes on version lower than 2024.5.0, confirmed that notifications and media player are working, then I upgraded to 2024.5.0 and tested the same - appears to work as expected.

image

I followed this and it worked. But in my case, the file "init.py" is called "init.py" so I tried to rename it to "init.py" and do the changes+restart and it didn't worked, then I rename it to "__init.py" with the changes +restart and it worked. No idea why mine has a diferent name, but it is working. Thanks for the backup tip @Protoncek and thanks for the guide/solution @atax112

zanix commented 2 months ago

Since author of this addon doesn't actively work on this (latest version is 2022) i think that probably doesn't read these issues also. So, i found a solution on a more recent fork of hass agent: https://github.com/hass-agent/HASS.Agent-Integration/issues/2 After this manual modification all works again.

Hey, I guess the link is wrong? Doesnt point to what you mean... Edit: the text was ok, the link not... hass-agent#2

Ok, so, quick&dirty fix for anyone that can/wants edit the files manually :D

Navigate to "/config/custom_components/hass_agent" - either inside the container or via some editor when using the supervisor enabled HA. Edit "init.py" file (for example "vi init.py") Add "async" in following places: a) line 27: "async def update_device_info(hass: HomeAssistant, entry: ConfigEntry, new_device_info):" b) line 139: "async def updated(message: ReceiveMessage):" Add "await" in following places: a) line 144: "await update_device_info(hass, entry, payload)" Restart Home Assistant What I did was do the above changes on version lower than 2024.5.0, confirmed that notifications and media player are working, then I upgraded to 2024.5.0 and tested the same - appears to work as expected.

image

I followed this and it worked. But in my case, the file "init.py" is called "init.py" so I tried to rename it to "init.py" and do the changes+restart and it didn't worked, then I rename it to "__init.py" with the changes +restart and it worked. No idea why mine has a diferent name, but it is working. Thanks for the backup tip @Protoncek and thanks for the guide/solution @atax112

The fork has these fixes, it requires removing this integration and adding the new one. https://github.com/hass-agent/HASS.Agent-Integration

formatBCE commented 2 months ago

Not only fork is fixed, but also actively supported, seems like. Moving to it.