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

[netatmo] Enforce API reconnect delay #18029

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

clinique
Copy link
Contributor

@clinique clinique commented Jan 3, 2025

Resolves #16485

@clinique clinique requested a review from lolodomo as a code owner January 3, 2025 16:57
@jlaur jlaur added the additional testing preferred The change works for the pull request author. A test from someone else is preferred though. label Jan 4, 2025
@jlaur
Copy link
Contributor

jlaur commented Jan 4, 2025

@clinique - thanks! I added the "additional testing preferred label" based on #16485 (comment). Let me know when you've had time to test it, then I'll also give it a test run.

@jlaur
Copy link
Contributor

jlaur commented Jan 4, 2025

Regression of #16489 which doesn't effectively block requests after status code 429.

@jlaur jlaur added bug An unexpected problem or unintended behavior of an add-on regression Regression that happened during the development of a release. Not shown on final release notes. labels Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
additional testing preferred The change works for the pull request author. A test from someone else is preferred though. bug An unexpected problem or unintended behavior of an add-on regression Regression that happened during the development of a release. Not shown on final release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[netatmo] Refresh misbehavior
3 participants