Skip to content

Comments

Fix Lutron Z3-1BRL (Aurora) actions#11317

Open
pikeas wants to merge 2 commits intoKoenkk:masterfrom
pikeas:patch-2
Open

Fix Lutron Z3-1BRL (Aurora) actions#11317
pikeas wants to merge 2 commits intoKoenkk:masterfrom
pikeas:patch-2

Conversation

@pikeas
Copy link
Contributor

@pikeas pikeas commented Jan 17, 2026

Switch now sends on_press | off_press | rotate:

{"action":"rotate","battery":100,"brightness":205,"linkquality":91,"update": ...}
{"action":"off_press","action_group":8093,"battery":100,"brightness":197,"linkquality":91,"update": ...}
{"action":"on_press","action_group":8093,"battery":100,"brightness":197,"linkquality":91,"update": ...}

We could drop brightness from the press payload if that's more consistent with other devices.

Separately, we can distinguish off/on based on what the switch reports, but...should we? Or should we combine them into a stateless press? Eg, if a light changes state because of an outside service call, it becomes out of sync with this switch. This can easily be handled at the automation level, but perhaps it should be handled here.

Related: Koenkk/zigbee2mqtt#28450, Koenkk/zigbee2mqtt#25538, Koenkk/zigbee2mqtt#12291

@pikeas
Copy link
Contributor Author

pikeas commented Jan 17, 2026

After making this PR, I realized that for my own use, I prefer the actions to be press | rotate_left | rotate_right, because switch state isn't coupled to light state. This aligns with how other switches behave but would be very different from how the current implementation works.

@Koenkk
Copy link
Owner

Koenkk commented Jan 18, 2026

This is a breaking change, so will merge this with z2m 3.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants