Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Limit api usage #145

Closed
wants to merge 2 commits into from
Closed

Conversation

derjoerg
Copy link
Collaborator

@derjoerg derjoerg commented Jan 3, 2025

If the state of a dimming_actuator or switch_actuator is already in the desired state (e.g. execute turn_off() and device is already turned off) no call is placed on the API.

Closes #144

@derjoerg derjoerg requested a review from kingsleyadam as a code owner January 3, 2025 13:36
@kingsleyadam
Copy link
Owner

I'm not convinced this is the best way to handle the root issue of 429 throttling requests. See my comment here: kingsleyadam/local-abbfreeathome-hass#117 (comment)

@derjoerg
Copy link
Collaborator Author

derjoerg commented Jan 7, 2025

As discussed in kingsleyadam/local-abbfreeathome-hass#117.

A more general approach will be implemented using "retry with exponential backoff".

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.

Don't send command if device is already in desired state
2 participants