Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@liudger
Copy link
Contributor

@liudger liudger commented Nov 15, 2025

Proposed change

This pull request adds documentation for a new action to the BSB-Lan integration, allowing users to synchronize the time on their BSB-Lan devices with Home Assistant's system time. This helps ensure accurate timestamps for device logs and schedules.

New action documentation:

  • Added a section describing the bsblan.sync_time action, including its purpose, usage, and configuration options.
  • Provided example YAML snippets for syncing time for all devices, a specific device, and scheduling the action in an automation.

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

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.

@home-assistant home-assistant bot added has-parent This PR has a parent PR in another repo next This PR goes into the next branch labels Nov 15, 2025
@netlify
Copy link

netlify bot commented Nov 15, 2025

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 4854cd1
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/6918973d2727ab00080564b9
😎 Deploy Preview https://deploy-preview-41789--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@liudger liudger marked this pull request as ready for review November 15, 2025 15:09
Copilot AI review requested due to automatic review settings November 15, 2025 15:09
Copilot finished reviewing on behalf of liudger November 15, 2025 15:13
Copy link
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 pull request adds documentation for the new bsblan.sync_time action to the BSB-Lan integration. The action enables users to synchronize their BSB-Lan device's time with Home Assistant's system time, helping maintain accurate timestamps for heating system logs and schedules.

Key changes

  • Added an "Actions" section to document the integration's available actions
  • Documented the bsblan.sync_time action with clear descriptions, configuration options, and data attributes
  • Provided three practical YAML examples: basic usage, device-specific sync, and automated daily synchronization


{% configuration_basic %}
Target:
description: Select the BSB-Lan device to sync. If no device is selected, all BSB-Lan devices will be synchronized.
Copy link

Copilot AI Nov 15, 2025

Choose a reason for hiding this comment

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

On line 68, the phrase "all BSB-Lan devices will be synchronized" could be clearer. According to the documentation guidelines, we should write from a second-person perspective using "you" and "your" rather than passive voice where possible.

Consider rephrasing to: "If you don't select a device, Home Assistant synchronizes all your BSB-Lan devices." or "If no device is selected, Home Assistant will sync time for all your BSB-Lan devices."

Copilot generated this review using guidance from repository custom instructions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-parent This PR has a parent PR in another repo next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant