search

konnected-io / konnected-esphome

ESPHome firmware configuration and recipes for Konnected devices
Other
50 stars 31 forks source link

Latest ESPHome reserves id 'alarm', causing build errors #12

Closed depoll closed 1 year ago

depoll commented 1 year ago

ESPHome 2023.6.4 has started erroring on builds with the following:

ID 'alarm' is reserved internally and cannot be used.
  id: alarm
  name: Alarm
  pin: D8
  platform: gpio
  icon: mdi:bullhorn
  internal: true
heythisisnate commented 1 year ago

Thank you for reporting. This caught me by surprise and will, unfortunately, require a breaking change on our side. Related to: https://github.com/esphome/esphome/pull/5042

I think the simple fix is to rename this id to siren. Any other thoughts?

depoll commented 1 year ago

Seems reasonable to me. I wish there were an easier way to roll out breaking changes to these packages, but barring a more holistic solution, it seems worth at least getting things into a healthy state again.

si458 commented 1 year ago

just picked up my esp8266 and same issue here but im not intrested in the siren/alarm, just the PIRs and door sensors

INFO ESPHome 2023.6.4
INFO Reading configuration /config/esphome/alarm-panel-6a4ee8.yaml...
Failed config

switch.gpio: [source /config/esphome/.esphome/packages/d9505d5d/packages/alarm-panel/alarm.yaml:3]

  ID 'alarm' is reserved internally and cannot be used.
  id: alarm
  name: Alarm
  pin: D8
  platform: gpio
  icon: mdi:bullhorn

FEATURE REQUEST: allow us to disable which features/zones we dont want..

heythisisnate commented 1 year ago

@si458 you can customize the features you want by editing the config in ESPHome. Replace the default package with the list of packages you need. See: https://support.konnected.io/esphome-on-the-konnected-alarm-panel-esp8266

si458 commented 1 year ago

thanks @heythisisnate! i looked all over the support page and didnt find that link anywhere! it was hidden under the 'see more' option of the esphome section!