Skip to content

Fix icon mapping for phenomena 40#72

Merged
caiosweet merged 2 commits intomainfrom
codex/update-phenomena_icon-in-api.py
Jan 9, 2026
Merged

Fix icon mapping for phenomena 40#72
caiosweet merged 2 commits intomainfrom
codex/update-phenomena_icon-in-api.py

Conversation

@caiosweet
Copy link
Owner

Motivation

  • The PHENOMENA_ICON mapping contained two adjacent string literals for key 40, which results in accidental concatenation and an invalid icon value.
  • Using a single valid Material Design Icon string prevents display and parsing issues in the integration.
  • The change aims to ensure consistent icon lookups for the MARI/sea phenomena.

Description

  • Updated the PHENOMENA_ICON mapping in custom_components/dpc/api.py to use a single icon for key 40.
  • Replaced the concatenated value "mdi:wave" "mdi:sail-boat" with the single icon "mdi:wave" for key 40 in PHENOMENA_ICON.
  • Performed a source inspection of the PHENOMENA_ICON dict to ensure there are no other accidental string concatenations.

Testing

  • No automated tests were run for this change.

Codex Task

@caiosweet caiosweet merged commit ff365fe into main Jan 9, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant