Skip to content

Add Gaposa brand assets#10148

Open
mwatson2 wants to merge 1 commit intohome-assistant:masterfrom
mwatson2:gaposa
Open

Add Gaposa brand assets#10148
mwatson2 wants to merge 1 commit intohome-assistant:masterfrom
mwatson2:gaposa

Conversation

@mwatson2
Copy link
Copy Markdown

@mwatson2 mwatson2 commented Apr 15, 2026

Proposed change

Adds the brand assets (icon and logo, normal and hDPI variants) for the new Gaposa integration. The artwork is derived from the official Gaposa SVG logo at https://www.gaposa.it/res/ftpgaposa/assets/images/logo-n.svg — the circular emblem on the left of the source SVG is used for the icon, and the emblem plus the "GAPOSA" wordmark is used for the logo. All four PNGs are transparent, interlaced, and dimensionally compliant with the brands repository validator.

Type of change

  • Add a new logo or icon for a new core integration
  • Add a missing icon or logo for an existing core integration
  • Replace an existing icon or logo with a higher quality version
  • Replace an existing icon or logo after a branding change
  • Removing an icon or logo

Additional information

Checklist

  • The added/replaced image(s) are PNG
  • Icon image size is 256x256px (icon.png)
  • hDPI icon image size is 512x512px for (icon@2x.png)
  • Logo image size has min 128px, but max 256px, on the shortest side (logo.png)
  • hDPI logo image size has min 256px, but max 512px, on the shortest side (logo@2x.png)

mwatson2 added a commit to mwatson2/core that referenced this pull request Apr 16, 2026
Before: the four bronze-tier rules that depend on external PRs
(`brands`, `docs-high-level-description`, `docs-installation-instructions`,
`docs-removal-instructions`) were marked as `todo` with the
rationale that they are only truly "done" once the sibling PRs
merge. This was causing the `Check hassfest` CI job to fail with:

    Quality scale tier bronze requires quality scale rules to be met:
      brands: todo
      docs-high-level-description: todo
      docs-installation-instructions: todo
      docs-removal-instructions: todo

Checking recent merged new-integration PRs shows the actual
convention is to flip these to `done` in the core PR as soon as
the sibling brands and docs PRs are open and under review. Two
concrete recent examples:

- home-assistant#167220 "Add Duco integration" (merged 2026-04-09 21:54 UTC)
  had all four rules as `done` in its quality_scale.yaml at the
  merge commit e7e4c49. The sibling brands PR home-assistant/brands#10068
  merged ~7 hours later at 2026-04-10 04:50 UTC, and the docs PR
  home-assistant.io#44468 merged ~9 hours after at 2026-04-10 07:09 UTC.
- home-assistant#166923 "Add Denon rs232 integration" (merged 2026-04-16 16:23 UTC)
  also had all four as `done`. The brands PR home-assistant/brands#10058
  merged 20 seconds later at 16:24 UTC, and the docs PR
  home-assistant.io#44437 is still OPEN at time of this commit —
  i.e. the core PR merged before the docs PR even merged.

The sibling PRs for gaposa are open and linked from the main PR:
- brands PR: home-assistant/brands#10148 (asset upload passed validate)
- docs PR:   home-assistant/home-assistant.io#44795 (textlint clean,
             Copilot review addressed)

Following the same convention here so hassfest passes and the core
PR can proceed through review in parallel with the sibling PRs.
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.

1 participant