benct / lovelace-multiple-entity-row

Show multiple entity states and attributes on entity rows in Home Assistant's Lovelace UI
MIT License
768 stars 55 forks source link

As of 2023.4.3 the m-e-row shows status of people as raw home/not_home instead of Home/Away #292

Closed wigster closed 1 year ago

wigster commented 1 year ago

After upgrade to 2023.4.3 from 2023.4.2, the status of people is being shown as home/not_home (the real state string) and not the friendly version.

image

aksnet commented 1 year ago

This seems to be the same issue for battery values, see: When custom:multiple-entity-row is used the value switches to english instead of german. 2023-04-13 13_26_14-Übersicht – Home Assistant

ildar170975 commented 1 year ago

Confirmed, 2023.4.4 (worked fine in 2023.4.0)

Something changed in HA suddenly w/o reporting as a "breaking change".

All values are displayed w/o localization: https://community.home-assistant.io/t/2023-4-custom-template-macros-and-many-more-new-entity-dialogs/556528/516?u=ildar_gabdullin image

type: entities
entities:
  - type: custom:multiple-entity-row
    entity: person.ildar
  - type: custom:multiple-entity-row
    entity: binary_sensor.testing_device_class_connectivity
  - type: custom:multiple-entity-row
    entity: sun.sun
piitaya commented 1 year ago

With 2023.4, HA core added support for translations attributes and name so entity state translation keys have changed too. The computeStateDisplay in this repository is outdated and need to be updated with new translation keys.

https://github.com/home-assistant/frontend/blob/dev/src/common/entity/compute_state_display.ts

BTW, it's a duplicate of https://github.com/benct/lovelace-multiple-entity-row/issues/290

Anton-Ka commented 1 year ago

В версии 2023.4 получена высокая доступность высокой скорости записи и имени, поэтому ключи записи состояния объекта также изменились. ComputeStateDisplay в этом репозитории его устарел, и необходимо обновить с помощью новых клавиш.

https://github.com/home-assistant/frontend/blob/dev/src/common/entity/compute_state_display.ts

Should something be done about this, or will it be fixed in an update?

sheminasalam commented 1 year ago

Problem persists with my door binary sensors even after updates. image

SAOPP commented 1 year ago

This issue is present everywhere, where I use binary sensors and m-e-row card.

alexruffell commented 1 year ago

Same here! I use this extensively on my dashboards so now they are all broken :(

If HA had the default feature of allowing to put a battery icon and battery % next to the sensor's reading, I wouldn't need it so much... hint hint ;)

image
Fierys commented 1 year ago

Yes, it doesn't look very nice. what to do with it? should i wait for the update? image_2023-04-16_20-31-45

mill1000 commented 1 year ago

With 2023.4, HA core added support for translations attributes and name so entity state translation keys have changed too. The computeStateDisplay in this repository is outdated and need to be updated with new translation keys.

https://github.com/home-assistant/frontend/blob/dev/src/common/entity/compute_state_display.ts

@piitaya I've seen similar issues in other custom cards due to out-of-date or homebrewed localization. Is there anyway to access the computeStateDisplay from the frontend itself?

sheminasalam commented 1 year ago

any updates

GRClark commented 1 year ago

Is there any progress on this? Love the integration but not sure whether I should bite the bullet upgrading and change all my dashboard code or hold off in hopes this will be fixed.

GRClark commented 1 year ago

@sheminasalam

any updates

There’s a pull request but that was a month ago. Not sure how long it takes to become a published update.

https://github.com/benct/lovelace-multiple-entity-row/pull/293

mill1000 commented 1 year ago

Those looking for a quick solution can add my fork as a custom repo under HACs. See here: https://github.com/benct/lovelace-multiple-entity-row/pull/293#issuecomment-1559725690

alexruffell commented 1 year ago

@mill1000 Thank you for sharing and making it easy for all to fix the issue while we wait on the original developer/maintainer to merge your PR.

GRClark commented 1 year ago

Those looking for a quick solution can add my fork as a custom repo under HACs. See here: #293 (comment)

@mill1000 should we remove the previous multiple entity row integration from HACS before/also?

Really appreciation you doing this and coming to the rescue!

alexruffell commented 1 year ago

@GRClark I left everything as it was and simply added the new repository. As soon as I did that, the updated integration appeared in a green box and I hit download. Problem was fixed. I left the original one in case the original developer resumes updating it.

benct commented 1 year ago

Pushed a new release containing the fix from @mill1000, as well as some other PRs. I'm not on the latest HA yet, so let me know if something is wrong.

SAOPP commented 1 year ago

@benct working as expected! Thanks homie!

Anton-Ka commented 1 year ago

@benct working as expected! Thanks homie!

i confirm this.