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

Changes to config_entry in option flow #535

Closed
cptshrk108 opened this issue Jan 8, 2025 · 5 comments · Fixed by #536
Closed

Changes to config_entry in option flow #535

cptshrk108 opened this issue Jan 8, 2025 · 5 comments · Fixed by #536

Comments

@cptshrk108
Copy link
Contributor

Version of the custom_component

2024.12.1

Describe the bug

En voulant apporter des modifications à l'intégration, j'ai réaliser que je ne pouvais pas accéder aux configurations. Ça me donnait une erreur dans le UI:

Config flow could not be loaded: 500 Internal Server Error Server got itself in trouble"

Dans le log on retrouve entre autre :

AttributeError: 'HiloOptionsFlowHandler' object has no attribute 'config_entry'. Did you mean: '_config_entry'?

En investigant je suis tombé sur cette PR de HACS : hacs/integration#4181

J'ai modifié le HiloOptionsFlowHandler init pour que ça match le changement dans HACS et ça a réglé le problème:

from awesomeversion import AwesomeVersion
from homeassistant.const import __version__ as HAVERSION
class HiloOptionsFlowHandler(config_entries.OptionsFlow):
    """Handle a Hilo options flow."""
    def __init__(self, config_entry: ConfigEntry) -> None:
        """Initialize"""
        if AwesomeVersion(HAVERSION) < "2024.11.99":
            self.config_entry = config_entry

Je ferai pas de PR parce que je comprend pas toute le code, mais bon je voulais le remonter si quelqu'un a le même issue.

@ic-dev21
Copy link
Collaborator

ic-dev21 commented Jan 8, 2025

Salut, quelle version de HA?

À partir de 2024.12.X on a un warning que d'adresser le config flow explicitement est deprecated et va être retiré fin 2025.

Voir #520

#521

J'aurais du en effet rajouter un version check comme tu l'as fait.

Te gènes pas de faire un PR au besoin :)

@cptshrk108
Copy link
Contributor Author

Bizarre que ma version de HA devrait pas avoir de issue avec ça:

Core 2024.11.2
Supervisor 2024.12.3
Operating System 13.2

@ic-dev21
Copy link
Collaborator

ic-dev21 commented Jan 8, 2025

Bizarre que ma version de HA devrait pas avoir de issue avec ça:

Core 2024.11.2 Supervisor 2024.12.3 Operating System 13.2

Version de core est plus basse que 2024.12, si tu updates d'après-moi le warning saute.

P.S. Je comprends pas toute le code non plus ;)

@cptshrk108
Copy link
Contributor Author

Bizarre que ma version de HA devrait pas avoir de issue avec ça:
Core 2024.11.2 Supervisor 2024.12.3 Operating System 13.2

Version de core est plus basse que 2024.12, si tu updates d'après-moi le warning saute.

P.S. Je comprends pas toute le code non plus ;)

Ouin c'était pas un warning though vraiment une erreur. Je vais upgrade mon HA demain quand y'aurait pas de défis!

Je vais pousser une PR avec mon changement aussi!

@ic-dev21
Copy link
Collaborator

ic-dev21 commented Jan 8, 2025

Je vais pousser une PR avec mon changement aussi!

That's the spirit!

Je m'étais mal exprimé mais en gros, si tu ne changeais pas le code et que tu upgradais à Core 2024.12 + tu n'aurais ni erreur ni warning.

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 a pull request may close this issue.

2 participants