Skip to content

Change Ring binary_sensor polling frequency to avoid rate limit exceeded errors#15414

Merged
fabaff merged 1 commit intohome-assistant:devfrom
tchellomello:fix_14693
Jul 11, 2018
Merged

Change Ring binary_sensor polling frequency to avoid rate limit exceeded errors#15414
fabaff merged 1 commit intohome-assistant:devfrom
tchellomello:fix_14693

Conversation

@tchellomello
Copy link
Copy Markdown
Contributor

@tchellomello tchellomello commented Jul 11, 2018

Description:

This PR changes the Ring binary_sensor polling frequency to avoid rate limit exceeded errors.

Many thanks to @dshokouhi, @Bergasha, and @JerryWorkman for testing and reports. 👍

Related issue (if applicable): fixes #14693

Example entry for configuration.yaml (if applicable):

ring:
  username: foo
  password: bar

binary_sensor:
   - platform: ring

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

If the code communicates with devices, web services, or third-party tools:

  • New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
  • New dependencies are only imported inside functions that use them ([example][ex-import]).
  • New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
  • New files were added to .coveragerc.

If the code does not interact with devices:

  • Tests have been added to verify that the new code works.

Copy link
Copy Markdown
Member

@fabaff fabaff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐦

@fabaff fabaff merged commit 6197fe0 into home-assistant:dev Jul 11, 2018
@ghost ghost removed the in progress label Jul 11, 2018
@tchellomello tchellomello deleted the fix_14693 branch July 11, 2018 15:16
awarecan pushed a commit to awarecan/home-assistant that referenced this pull request Jul 16, 2018
@balloob balloob mentioned this pull request Jul 20, 2018
michaeldavie pushed a commit to michaeldavie/home-assistant that referenced this pull request Jul 31, 2018
girlpunk pushed a commit to girlpunk/home-assistant that referenced this pull request Sep 4, 2018
@home-assistant home-assistant locked and limited conversation to collaborators Dec 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ring Binary Sensor issue

3 participants