[PR #2] [MERGED] Fix for current ESPHome version syntax #2

Closed
opened 2026-03-20 17:22:12 +01:00 by sascha_hemi · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/sascha-hemi/esphome-ikea-led-matrix/pull/2
Author: @leandroebner
Created: 4/7/2024
Status: Merged
Merged: 9/9/2024
Merged by: @sascha-hemi

Base: masterHead: master


📝 Commits (4)

  • 9fccfff fix compilation error if display is trying to register as component
  • 01cd3b0 DisplayBuffer now automatically inherits from PollingComponent
  • 1660e86 Update display.py
  • 5cd045a Merge pull request #1 from leandroebner/leandroebner-patch-1

📊 Changes

2 files changed (+9 additions, -1 deletions)

View changed files

📝 components/frekvens_panel/display.py (+2 -1)
📝 components/frekvens_panel/frekvens-panel.h (+7 -0)

📄 Description

Hi, the current ESPHome version requires different syntax for the display component. To be able to use this script independent from the version, an added library checks the current utilized ESPHome version and the syntax get's adapted to the necessary form.

I only fixed it for the Ikea Frekvens Panel, due the reason that's the only hardware I can verify my working code with. The procedure for the Obegränsad should be the same.

photo_2024-04-07_23-54-15


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/sascha-hemi/esphome-ikea-led-matrix/pull/2 **Author:** [@leandroebner](https://github.com/leandroebner) **Created:** 4/7/2024 **Status:** ✅ Merged **Merged:** 9/9/2024 **Merged by:** [@sascha-hemi](https://github.com/sascha-hemi) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (4) - [`9fccfff`](https://github.com/sascha-hemi/esphome-ikea-led-matrix/commit/9fccfff1abf3e7078c8c5455a77859fe083c9440) fix compilation error if display is trying to register as component - [`01cd3b0`](https://github.com/sascha-hemi/esphome-ikea-led-matrix/commit/01cd3b0a4d4099213e583e4833258df341a4e9ad) DisplayBuffer now automatically inherits from PollingComponent - [`1660e86`](https://github.com/sascha-hemi/esphome-ikea-led-matrix/commit/1660e865a476994809b8ccb037f0b0bde44fc4e1) Update display.py - [`5cd045a`](https://github.com/sascha-hemi/esphome-ikea-led-matrix/commit/5cd045ae9b64cca00bf655ef48ad5a2b27a50f73) Merge pull request #1 from leandroebner/leandroebner-patch-1 ### 📊 Changes **2 files changed** (+9 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `components/frekvens_panel/display.py` (+2 -1) 📝 `components/frekvens_panel/frekvens-panel.h` (+7 -0) </details> ### 📄 Description Hi, the current ESPHome version requires different syntax for the display component. To be able to use this script independent from the version, an added library checks the current utilized ESPHome version and the syntax get's adapted to the necessary form. I only fixed it for the Ikea Frekvens Panel, due the reason that's the only hardware I can verify my working code with. The procedure for the Obegränsad should be the same. ![photo_2024-04-07_23-54-15](https://github.com/sascha-hemi/esphome-ikea-led-matrix/assets/116027657/a48be619-610c-4acf-b79d-c3e720e868d2) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
sascha_hemi added the pull-request label 2026-03-20 17:22:12 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sascha_hemi/esphome-ikea-led-matrix#2