Skip to content

Add local brand icons#1050

Merged
bouwew merged 2 commits intomainfrom
local_brand
Mar 23, 2026
Merged

Add local brand icons#1050
bouwew merged 2 commits intomainfrom
local_brand

Conversation

@bouwew
Copy link
Contributor

@bouwew bouwew commented Mar 23, 2026

Summary by CodeRabbit

  • Documentation
    • Updated changelog to reflect upcoming beta brand icons feature.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 23, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 79bac324-6d23-4cc3-af69-352b47f8a1c5

📥 Commits

Reviewing files that changed from the base of the PR and between 372c848 and ab5c7c2.

⛔ Files ignored due to path filters (4)
  • custom_components/plugwise/brand/icon.png is excluded by !**/*.png
  • custom_components/plugwise/brand/icon@2x.png is excluded by !**/*.png
  • custom_components/plugwise/brand/logo.png is excluded by !**/*.png
  • custom_components/plugwise/brand/logo@2x.png is excluded by !**/*.png
📒 Files selected for processing (1)
  • CHANGELOG.md

📝 Walkthrough

Walkthrough

A single changelog entry was added to the "Ongoing" section documenting the addition of local beta brand icons, referencing the associated pull request.

Changes

Cohort / File(s) Summary
Documentation
CHANGELOG.md
Added one new bullet point under the "Ongoing" section referencing the local beta brand icons feature.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • CoMPaTech
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title 'Add local brand icons' accurately describes the main change—adding local beta brand icons as documented in the CHANGELOG.md update.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch local_brand

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai bot requested a review from CoMPaTech March 23, 2026 18:37
@sonarqubecloud
Copy link

@bouwew bouwew marked this pull request as ready for review March 23, 2026 18:57
@bouwew bouwew requested a review from a team as a code owner March 23, 2026 18:57
Copilot AI review requested due to automatic review settings March 23, 2026 18:57
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

Adds local branding assets for the plugwise custom component so Home Assistant can display integration icons/logos without relying on external brand sources.

Changes:

  • Added brand/icon.png and brand/icon@2x.png.
  • Added brand/logo.png and brand/logo@2x.png.
  • Updated CHANGELOG.md to mention the new local beta brand icons.

Reviewed changes

Copilot reviewed 1 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
custom_components/plugwise/brand/icon.png Adds 1x integration icon asset.
custom_components/plugwise/brand/icon@2x.png Adds 2x integration icon asset.
custom_components/plugwise/brand/logo.png Adds 1x integration logo asset.
custom_components/plugwise/brand/logo@2x.png Adds 2x integration logo asset.
CHANGELOG.md Notes the new local beta brand icons in the “Ongoing” section.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@CoMPaTech
Copy link
Member

@bouwew bouwew merged commit cfeca9b into main Mar 23, 2026
17 checks passed
@bouwew bouwew deleted the local_brand branch March 23, 2026 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants