Mismatched sensor naming between foxess_modbus integration and this lovelace card

shiraz-b commented 10 months ago

Version of the custom_component

v1.0.1

Configuration

Installed and successfully running https://github.com/nathanmarlor/foxess_modbus [v1.7.2](https://github.com/nathanmarlor/foxess_modbus/releases/tag/v1.7.2)

Describe the bug

Installed and successfully ran https://github.com/nathanmarlor/foxess_modbus v1.7.2
During the above installation chose a friendly name of "foxess 485 modbus"
This resulted in sensors being created without the friendly name in the sensor name itself, but in the display name - not a problem in and of itself - everything works really well and I can see the data from the Invertor.
Installed this lovelace card
Setup the card in a Dashboard
The dashboard fails with the following errors:

I suspect this is a mismatch issue. The lovelace card is expecting the sensor name to have the defined friendly name to be prepended to the sensor names, but the integration doesn't do this.

This is all great work and I'm happy to help in anyway I can with any beta versions/reviews etc - just let me know, thanks!

Debug log

N/A

canton7 commented 10 months ago

Looks like the same as #8. You've misdiagnosed the issue slightly - the entity ids are supplied by the integration, but the entities don't have the id in HA that the integration specified. Either that, or some thing's broken.

Thanks for the setup steps - I'll try and repro over the weekend.

canton7 commented 10 months ago

Can you give v1.7.3b1 of the foxess_modbus integration a try please?

shiraz-b commented 10 months ago

Thanks @canton7 : I'll give it a try today and confirm the outcome...

shiraz-b commented 10 months ago

Hi @canton7 : I can confirm after deleteing/reloading both the main integration to v1.7.3b1 and the lovelace integration (probably unecessarily) it all works. Specifically:

the EntityId's are now prefixed with the chosen name.
the card is reading/writing to the integration correctly (confirmed by checking the foxesscloud battery settings after changing the lovelace UI)

Thank you so much!!

I'll stick with the beta until it's rolled into the main line tags...

Not sure of the process here - happy to close if you'd like?

canton7 commented 10 months ago

Thanks! I'll get the beta released, then close this.

canton7 commented 10 months ago

Released

nathanmarlor / foxess_modbus_charge_period_card