Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Prefer head: over entity: #231

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ajma
Copy link

@ajma ajma commented Sep 28, 2022

When both entity: and head: are included, entity: will supersede head:. Some components (like auto-entities) will automatically set entity: and so any head: options will get ignored.

Use case:

type: custom:auto-entities
card:
  type: entities
filter:
  include:
    - group: light.downstairs
      options:
        type: custom:fold-entity-row
        head:
          entity: this.entity_id
          secondary_info: last-changed

Fixes #230

When both entity: and head: are included, entity: will supersede head:. Some components (like auto-entities) will automatically set entity: and so any head: options will get ignored.

Use case:
```
type: custom:auto-entities
card:
  type: entities
filter:
  include:
    - group: light.downstairs
      options:
        type: custom:fold-entity-row
        head:
          entity: this.entity_id
          secondary_info: last-changed
```
@ajma ajma changed the title Prefer head: over entity: fix: Prefer head: over entity: Oct 25, 2022
@ajma
Copy link
Author

ajma commented Oct 31, 2022

@thomasloven any thoughts on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't include head: options when using inside a auto-entities group filter
1 participant