Baldhor / Homey-ESPhome-Enhanced

This is an Homey app which adds support for ESPHome devices. ESPHome is a system to control your ESP8266/ESP32 by simple yet powerful configuration files and control them remotely through Home Automation systems. More information on their homepage: https://www.esphome.io/
GNU General Public License v3.0
5 stars 3 forks source link

v1.0.3 - Wizard - When Main menu opens, error "Action failed with message: Invalid capability: text" pops up #48

Closed PeterKawa closed 1 year ago

PeterKawa commented 1 year ago

Describe the bug

  1. After starting the wizard when the Main menu appears, this pop up appears: "Action failed with message: Invalid capability: text". I can hit "OK" and nothing seems to be affected.

To Reproduce Steps to reproduce the behavior:

  1. start the 'ESPhome wizard'
  2. when Main menu is in focus, error pops up

Expected behavior No error popping up ;-)

Screenshots signal-2023-10-03-163041_002

Desktop (please complete the following information):

Smartphone (please complete the following information):

Baldhor commented 1 year ago

@PeterKawa The capability "text" doesn't exist :) The closest would be the "custom_text" capability but it's broken, see issue #46

Possible to enable console.re and collect some logs so I can understand ?

Baldhor commented 1 year ago

https://github.com/Baldhor/Homey-ESPhome-Enhanced/wiki/ConsoleRe-support

Baldhor commented 1 year ago

Received a stack trace:

Error: Invalid Capability: text
    at Remote Process
    at HomeyClient.emit (/node_modules/@athombv/homey-apps-sdk-v3/lib/HomeyClient.js:44:23)
    at Object.emit (/node_modules/@athombv/homey-apps-sdk-v3/manager/drivers.js:78:54)
    at Object.emit (/node_modules/@athombv/homey-apps-sdk-v3/lib/Driver.js:126:50)
    at VirtualDevice.addCapability (/node_modules/@athombv/homey-apps-sdk-v3/lib/Device.js:446:25)
    at /app/drivers/esphome-wizard/driver.js:412:35
    at PairSession._triggerEvent (/node_modules/@athombv/homey-apps-sdk-v3/lib/PairSession.js:80:12)
    at ManagerDrivers._onPairSessionEvent (/node_modules/@athombv/homey-apps-sdk-v3/manager/drivers.js:190:20)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Baldhor commented 1 year ago

Ok actually, there was a 'text' capability lol It was supposed to be named 'esphome_text', hence the issue ...

Here is what I gonna do:

  1. I will remove from definition the existing 'custom' capabilities!
  2. When starting the application, I will remove from the existing virtual devices any 'text' or whatever custom capabilities!
  3. Through an enhancement I will add a lot of "well made" custom capabilities!

Tabula Rasa

Baldhor commented 1 year ago

@PeterKawa I published version 1.0.5 to solve the issue. I removed completely the text capability support, and at application restart it will remove it from existing virtual devices.

Yes, it's not really a solution, more likely a workaround, but I prefer to redo the custom capabilities from ground up (with icons, and so on) => refer to #41.

Can you confirm it fix your problems?

PeterKawa commented 1 year ago

Confirmed. It's fixed with v 1.0.5. (already mentioned via Homey forum). Thanks!

PeterKawa commented 1 year ago

Thankssssss! I believe I acked it's working @ forum topic. (Sorry I'm just not used to git issues and its updates, wasn't ignoring you 🤭 )

Op wo 4 okt. 2023 21:02 schreef Baldhor @.***>:

Closed #48 https://github.com/Baldhor/Homey-ESPhome-Enhanced/issues/48 as completed.

— Reply to this email directly, view it on GitHub https://github.com/Baldhor/Homey-ESPhome-Enhanced/issues/48#event-10554744163, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARUTUUC3S2R3BF3TDXMGVGDX5WXFNAVCNFSM6AAAAAA5REFTTGVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJQGU2TINZUGQYTMMY . You are receiving this because you were mentioned.Message ID: @.*** com>