Skip to content

Prevent PositionableAbility.targetPosition from being set to 0% or 100% unless the Shelly has been manually controlled#325

Open
NMAC427 wants to merge 3 commits intoalexryd:masterfrom
NMAC427:master
Open

Prevent PositionableAbility.targetPosition from being set to 0% or 100% unless the Shelly has been manually controlled#325
NMAC427 wants to merge 3 commits intoalexryd:masterfrom
NMAC427:master

Conversation

@NMAC427
Copy link

@NMAC427 NMAC427 commented Jul 22, 2021

After specifying a target position using the Home app, the slider would automatically jump to 0 or 100 (depending on wether it was moving up or down) until the Shelly stopped moving. This pull request changes this behavior. Now the target position only gets set to 0 / 100 if the user manually controls the Shelly using either the Shelly app or a physical button.

@NMAC427
Copy link
Author

NMAC427 commented Jul 22, 2021

The current debounce algorithm would prefer older target position updates over newer ones. This would lead to HomeKit getting out sync when changing the target position too quickly.

@sylwester-
Copy link

@alexryd - any chance that this could be merged? Simple change but helps a lot with consistency in Home app. Happy to help with testing if needed.

@NMAC427
Copy link
Author

NMAC427 commented Dec 21, 2021

@alexryd - Any updates?

@TheManchineel
Copy link

Almost one year later and this is not merged. This is a literal life-saver of a PR, it's needed if you want a good experience with Shelly roller shutters on HB. Any chance this might get merged? @alexryd

@VF-mbrauer
Copy link

@alexryd is that plugin still supported? If not then we would need to change for a one which still has a proper lifecycle. Thanks.

@TheManchineel
Copy link

Still no updates? Oh come on

@sylwester-
Copy link

Not sure if it helps but I moved to

https://github.com/mongoose-os-apps/shelly-homekit

Even though I have 12 Shelly 2.5 as shutters it was still worth it.

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.

4 participants