home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
72.81k stars 30.51k forks source link

iAqualink Light Issue #82227

Closed ghost closed 1 year ago

ghost commented 1 year ago

The problem

The November release broke functionality of the light entity for my iAqualink integration.

What version of Home Assistant Core has the issue?

core-2022-11.0

What was the last working version of Home Assistant Core?

core-2022-10.0

What type of installation are you running?

Home Assistant OS

Integration causing the issue

iAqualink

Link to integration documentation on our website

https://www.home-assistant.io/integrations/iaqualink/

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 428, in async_add_entities
    await asyncio.gather(*tasks)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 605, in _async_add_entity
    capabilities=entity.capability_attributes,
  File "/usr/src/homeassistant/homeassistant/components/light/__init__.py", line 929, in capability_attributes
    data[ATTR_EFFECT_LIST] = self.effect_list
  File "/usr/src/homeassistant/homeassistant/components/iaqualink/light.py", line 88, in effect_list
    return list(self.dev.supported_light_effects)
AttributeError: 'IaquaColorLightHU' object has no attribute 'supported_light_effects'

Additional information

No response

home-assistant[bot] commented 1 year ago

Hey there @flz, mind taking a look at this issue as it has been labeled with an integration (iaqualink) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `iaqualink` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Change the title of the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign iaqualink` Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


iaqualink documentation iaqualink source (message by IssueLinks)

markus99 commented 1 year ago

Dupe of this issue: https://github.com/home-assistant/core/issues/81818

flz commented 1 year ago

This should be fixed?

RPHllc commented 1 year ago

I can confirm that it is fixed, I can turn the pool lights on/off and select different color schemes. I am on HA version 2022.12.1

ghost commented 1 year ago

resolved

ghost commented 1 year ago

in the latest update this changed from a light to a switch... not sure if that was intentional

RPHllc commented 1 year ago

in the latest update this changed from a light to a switch... not sure if that was intentional

I continue to see it as a light. The code below works for me.

action:
      - service: light.turn_on
        target:
          entity_id: light.pool_light
        data:
          effect: '{{ [ "SAm", "Party", "Romance", "Caribbean", "American", 
                    "California Sunset", "Royal", "Blue", "Green", "Red",
                    "White", "Magenta" ] | random }}'