search

corporategoth / ha-powerpetdoor

Home Assistant plugin to enable control and monitoring of the Power Pet Door by High Tech Pet
MIT License
17 stars 4 forks source link

Unable to connect to power pet door / Validation of translation placeholders for localized (de) string component.homeassistant.issues.config_entry_reauth.title failed #13

Open udotrautmann opened 8 months ago

udotrautmann commented 8 months ago

After Update Home Assistant Core to 2024.2.4 and Operating System to 12.0 the ha-powerpetdoor plugin is not working anymore!

image

udotrautmann commented 8 months ago

I have downgraded to Core 2024.2.3 and it is working now

corporategoth commented 8 months ago

Interesting. The deprecated token is fine, that's already fixed, but I didn't push up a new version just for that, I have until 2025 for that.

However, Power Pet Door has no localizations, I only speak english and nobody has submitted translations. Despite this, component.homeassistant.issues.config_entry_reauth.title is a token I should not be defining anyway, this would be a missing translation token from home assistant itself.

This seems to be a bug in home assistant, not power pet door. https://github.com/home-assistant/core/issues/111621 https://github.com/home-assistant/core/issues/111561 https://github.com/home-assistant/core/issues/111562 https://github.com/home-assistant/core/issues/111516

Or at least, a new requirement that was never communicated that likely broke a bunch of plugins. I'll look into seeing if I need to do something about this.

corporategoth commented 8 months ago

https://github.com/home-assistant/core/pull/111275 seems to be the cause. It seems to be expecting some token to be defined. But is not declaring any default, nor do I know exactly where I am supposed to put said token.

corporategoth commented 8 months ago

I think I can see how to fix this. Apparently I need to add a title for the config screens.

corporategoth commented 8 months ago

Could you try this with the latest source in git (if you can pull from the git repo) - I updated the custom_components/powerpetdoor/translations/en.json to hopefully provide the missing token. If I get validation this now works with 2024.2.4 / 12.0 I will release a new version.

udotrautmann commented 8 months ago

sorry i am not familiar with github i am not a developer ;-) is there a other way for testing for example downloding and replace the file en.json manually?

corporategoth commented 8 months ago

Here is the raw file you could download:

https://raw.githubusercontent.com/corporategoth/ha-powerpetdoor/main/custom_components/powerpetdoor/translations/en.json

sdholden28 commented 8 months ago

I can find no issues with this integration, using the modified en.json file linked above. HA version: 2024.2.5 HAOS version: 12.0

It seems the HA issue was fixed in 2024.2.5 here: https://github.com/home-assistant/core/pull/111648

So perhaps the en.json is not needed any longer, but in any case, it doesn't seem to cause any issues on its own.