Fix pour issue #442 #443
Merged
Fix pour issue #442 #443
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Causé par un upstream PR dans le codebase de HA. Voir home-assistant/core##110685
|
Je vais essayer de rajouter une détection de version HA Core parce que si on utilise ça avec une version avant 2024.7 ça brise aussi |
Si on met à jour le custom component à 2024.7.1 sans avoir au moins HA 2024.7.X, le component brisait. Cette boucle if permet de conserver l'ancienne logique si la version est plus vieille. C'est pas joli mais ça marche.
FrancoLoco
approved these changes
Jul 3, 2024
FrancoLoco
approved these changes
Jul 3, 2024
valleedelisle
approved these changes
Jul 3, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Causé par un upstream PR dans le codebase de HA. Voir home-assistant/core#110685
Ajouté une entrée de 120 secondes comme max délai, après quoi l'intégrale sera calculée quand même.
Ajouté l'attribut dans le super.init() du energysensor.
Bump up version dans manifest.json. On est assez proche de juillet on va dire.