home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
73.32k stars 30.62k forks source link

Notify groups from UI not working #123558

Open renssies opened 2 months ago

renssies commented 2 months ago

The problem

Is was excited to see notify groups in the list of helpers in the UI on 2024.8. However the notify group helper seems to be broken?

When I tied to create a notify group from the UI the list of entities is empty and states "No matching entities found". While I don't have any notify entities, I do have a number of notify actions (previously called services).

If I manually enter these notify "actions" and create the group, then it is saved. However the group does not appear under the actions (previously called services), while groups made in yaml do appear under actions. Using the notify.notify action with the target being the group results in "Failed to perform the action notify.notify. Unknown error"

What version of Home Assistant Core has the issue?

core-2024.8.1

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Container

Integration causing the issue

No response

Link to integration documentation on our website

https://www.home-assistant.io/integrations/group

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

home-assistant[bot] commented 2 months ago

Hey there @home-assistant/core, mind taking a look at this issue as it has been labeled with an integration (group) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `group` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Renames the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign group` Removes the current integration label and assignees on the issue, add the integration domain after the command. - `@home-assistant add-label needs-more-information` Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue. - `@home-assistant remove-label needs-more-information` Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


group documentation group source (message by IssueLinks)

Barsonax commented 2 months ago

I can confirm iam seeing the same behavior here. Seems its simply not in a working state yet.

giinuu commented 2 months ago

Can also confirm this issue happens for me, 2024.8.1

No obvious log errors, no failing requests or JS console errors.

joostlek commented 2 months ago

Please note that it's intended to group notify entities and that the majority of notify implementations still use notify services. For the services you still have to use the YAML groups

Barsonax commented 2 months ago

Please note that it's intended to group notify entities and that the majority of notify implementations still use notify services. For the services you still have to use the YAML groups

Ok didn't knew that.

Seems like the mobile homeassistant app only exposes services and no entities. So I guess that's changing soonish?

joostlek commented 2 months ago

There has been no action taken yet to start the migration, but they will eventually be migrated to the new notify entities yes

giinuu commented 2 months ago

I wonder if it is currently worth linking a note to the dev blog into the help section around notify groups, I can certainly see why there is confusion around this currently with the same naming convention.

E.g. something placed in this section;

https://www.home-assistant.io/integrations/group#notify-entity-groups

About

https://developers.home-assistant.io/blog/2024/04/10/new-notify-entity-platform/

Morphy99 commented 1 month ago

There has been no action taken yet to start the migration, but they will eventually be migrated to the new notify entities yes

Is there any way to track progress on this as I'd like to be able to edit notify groups in the UI as well? I can't see any open issues on the companion github

oerix commented 3 weeks ago

Any update on this please? Kind of strange that this has been merged to stable build as incomplete.