Skip to content

Remove scan_interval warning in PVOutput docs#17355

Merged
frenck merged 1 commit into
home-assistant:currentfrom
hmmbob:patch-2
Apr 9, 2021
Merged

Remove scan_interval warning in PVOutput docs#17355
frenck merged 1 commit into
home-assistant:currentfrom
hmmbob:patch-2

Conversation

@hmmbob
Copy link
Copy Markdown
Contributor

@hmmbob hmmbob commented Apr 9, 2021

Proposed change

Removing the warning in PVOutput docs about setting the scan_interval to >60sec, as the default for this sensor is already set at 120sec. So by default, this sensor shouldn't bust API limits and scan_interval should not need to be adjusted.

https://github.com/home-assistant/core/blob/dev/homeassistant/components/pvoutput/sensor.py#L35

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

  • Link to parent pull request in the codebase:
  • Link to parent pull request in the Brands repository:
  • This PR fixes or closes issue:

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

@probot-home-assistant probot-home-assistant Bot added the current This PR goes into the current branch label Apr 9, 2021
Copy link
Copy Markdown
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

Thanks, @hmmbob 👍

@frenck frenck merged commit 948e304 into home-assistant:current Apr 9, 2021
@hmmbob hmmbob deleted the patch-2 branch April 9, 2021 08:37
RenierM26 pushed a commit to RenierM26/home-assistant.io that referenced this pull request Apr 9, 2021
RenierM26 added a commit to RenierM26/home-assistant.io that referenced this pull request Apr 9, 2021
RenierM26 pushed a commit to RenierM26/home-assistant.io that referenced this pull request Apr 9, 2021
RenierM26 added a commit to RenierM26/home-assistant.io that referenced this pull request Apr 9, 2021
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed current This PR goes into the current branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants