Skip to content

Conversation

@blacknell
Copy link
Contributor

@blacknell blacknell commented Nov 13, 2025

I've noticed that the weekly schedule for the Sonoff TRVZB is not marked as type config for the entityCategory.

This results in extra entities displaying in dashboards when they shouldn't since they aren't control entities. Moving them to the configuration section is where they should be rather than as every day controls.

Note I've amended the modern extend weeklySchedule which is currently only referenced by the Sonoff TRV but this should make sense as the default for any future variants.

@blacknell blacknell force-pushed the fix-trvzb-entity-category branch from 6e4ce5c to c60d3e0 Compare November 13, 2025 08:40
@blacknell blacknell marked this pull request as ready for review November 13, 2025 08:42
@Koenkk Koenkk merged commit cd7a210 into Koenkk:master Nov 14, 2025
3 checks passed
@Koenkk
Copy link
Owner

Koenkk commented Nov 14, 2025

thanks

@blacknell
Copy link
Contributor Author

@Koenkk - I've just installed and I don't understand why it's now showing up in diagnostics rather than configuration. The same change I made on the SNZB-02 in PR 10717 has worked.

Can you advise and I'll submit a new PR

@Koenkk
Copy link
Owner

Koenkk commented Dec 2, 2025

Could you provide the debug log when z2m starts such that we can see what is published?

@blacknell
Copy link
Contributor Author

Could you provide the debug log when z2m starts such that we can see what is published?

Here are the lines where I grep'd for one of my devices and schedule

sch.log

@Koenkk
Copy link
Owner

Koenkk commented Dec 3, 2025

Seems to be because of this: https://github.com/Koenkk/zigbee2mqtt/blob/110149fa20c743bbba540207778b05c5eb575cf8/lib/extension/homeassistant.ts#L1087, so HA doesn't support entity category config for sensors.

@blacknell
Copy link
Contributor Author

Seems to be because of this: https://github.com/Koenkk/zigbee2mqtt/blob/110149fa20c743bbba540207778b05c5eb575cf8/lib/extension/homeassistant.ts#L1087, so HA doesn't support entity category config for sensors.

So the schedule shouldn't be a sensor. I don't understand the weeklyschedule method

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

Successfully merging this pull request may close these issues.

2 participants