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.

Sign up for GitHub

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

Add documentation for ccl #36357

Open
wants to merge 7 commits into
base: next
Choose a base branch
from
Open

Add documentation for ccl #36357

Changes from 2 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
0b116a8
Add documentation for ccl
fkiscd Dec 14, 2024
86e5904
Update ccl.markdown
fkiscd Dec 14, 2024
09969b0
Merge branch 'next' into ccl
fkiscd Dec 27, 2024
0393ec8
Update ccl.markdown
fkiscd Dec 27, 2024
614d55f
Update ccl.markdown
fkiscd Jan 6, 2025
9505366
Update ccl.markdown
fkiscd Feb 3, 2025
8bf37cf
add configuration_basic
fkiscd Feb 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
41 changes: 41 additions & 0 deletions source/_integrations/ccl.markdown
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
---
title: CCL Electronics
description: Instructions on how to integrate your CCL Electronics devices into Home Assistant.
ha_category:
- Sensor
- Weather
ha_release: 2025.1
Copy link
Contributor

Choose a reason for hiding this comment

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

💡 Codebase verification

Remove Weather category from documentation

The integration is incorrectly categorized under "Weather". While it provides weather station data through sensors, it does not implement a weather platform or weather entity. The Weather category should be removed from ha_category as it may mislead users.

  • Remove "Weather" from ha_category in source/_integrations/ccl.markdown
🔗 Analysis chain

Verify Weather platform implementation

The integration is categorized under "Weather" but doesn't list "weather" in the platforms. If weather data is exposed, consider adding "weather" to ha_platforms.

Also applies to: 13-15

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Check if weather platform is implemented in the core PR
gh api repos/home-assistant/core/pulls/130281 | jq -r '.body' | grep -i 'weather'

Length of output: 194

Script:

#!/bin/bash
# Check if weather platform is implemented in the integration
fd "ccl" --type f --exec rg -l "weather|Weather" {}

# Check for weather entity implementation
ast-grep --pattern 'class $_(WeatherEntity)'

Length of output: 136

ha_iot_class: Local Push
ha_config_flow: true
ha_codeowners:
- '@fkiscd'
ha_domain: ccl
ha_platforms:
- sensor
- binary_sensor
---

The **CCL Electronics** {% term integration %} is a local-only {% term integration %} that reads weather data from CCL HA series weather station on the local network.

{% include integrations/config_flow.md %}

## Weather Station configuration

1. In the **Settings** → **Devices & services**, click **Add integration** and select **CCL Electronics** from the pop-up menu.
2. Follow the instructions and press **Submit** to generate the following details.
For example:
- Server IP: 192.168.1.154 (your Home Assistant IP address)
- Port: 42373 (fixed)
- Path: /54fa0b (random)
3. Open the WSLink app in your mobile device.
4. Connect your mobile device to your weather station in AP mode.
5. In the WSLink app, go to **Weather Server** → **Home Assistant**, and enter the Server IP, Port, and Path assigned by Home Assistant.
6. Save

## Sensors

One weather station included 3 components. Each of them represents a “device” in Home Assistant:

- Console & Sensor Array
- Other Sensors
- Status
Copy link
Contributor

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

Add details about available sensors and data

The sensors section would benefit from:

  • Description of what each component provides
  • List of available sensors and their units
  • Example values or ranges
  • Update frequency information

Loading