Skip to content

Battery triggers: fix examples#45516

Open
c0ffeeca7 wants to merge 4 commits into
currentfrom
battery-triggers-fix-examples
Open

Battery triggers: fix examples#45516
c0ffeeca7 wants to merge 4 commits into
currentfrom
battery-triggers-fix-examples

Conversation

@c0ffeeca7
Copy link
Copy Markdown
Contributor

Proposed change

Battery triggers: fix examples

  • to use 101 with the between threshold, as the between threshold is an exclusive range (upper and lower bound are not included)
  • The 0-100 range is not enforced, so it is possible to actually enter 101.

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: fixes #

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.

Copilot AI review requested due to automatic review settings May 22, 2026 12:48
@home-assistant home-assistant Bot added the current This PR goes into the current branch label May 22, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented May 22, 2026

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 335cfdf
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/6a11274bacd3810008533686
😎 Deploy Preview https://deploy-preview-45516--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the battery trigger documentation examples to correctly include 100% in between (“In range”) thresholds, which are documented as exclusive on both bounds.

Changes:

  • Updated between range examples to use an upper bound of 101 so that 100% is included in the effective range.
  • Adjusted the accompanying explanation text to reflect the intended “includes 100%” behavior.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
source/_triggers/battery.level_crossed.markdown Updates the “safe range” example to include 100% by setting value_max to 101 and revises the descriptive sentence.
source/_triggers/battery.level_changed.markdown Updates the “healthy charge range” example to include 100% by setting value_max to 101 and revises the descriptive sentence.

Comment thread source/_triggers/battery.level_crossed.markdown Outdated
Comment thread source/_triggers/battery.level_changed.markdown Outdated
c0ffeeca7 and others added 2 commits May 22, 2026 15:02
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Comment thread source/_triggers/battery.level_changed.markdown Outdated
Comment thread source/_triggers/battery.level_crossed.markdown Outdated
Co-authored-by: Rita C. <215712047+Laufmaschine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

current This PR goes into the current branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants