search

dvd-dev / hilo

Home Assistant Hilo Integration via HACS
MIT License
126 stars 26 forks source link

Handle utility_meter Tariffs list to always exist #392

Closed elafontaine closed 6 months ago

elafontaine commented 6 months ago

Hi all,

This should be fixing the problem where the utility meter setup could fail because the configuration for tariff wasn't being passed by default (at least an empty list).

This is untested as I do not have the setup to test it locally. I will try to adjust.

Fix #365

ic-dev21 commented 6 months ago

Veux-tu que je me charge du linting?

Je peux tester sans problème ici. Je ferai ça dans les prochains jours.

ic-dev21 commented 6 months ago

En test pour le week-end. @elafontaine as-tu des tests particuliers que tu voudrais que je fasse? Je viens de le pousser sur un clean install en dev, avec seulement "generate energy meters" coché. Autre chose?

elafontaine commented 6 months ago

En test pour le week-end. @elafontaine as-tu des tests particuliers que tu voudrais que je fasse? Je viens de le pousser sur un clean install en dev, avec seulement "generate energy meters" coché. Autre chose?

Essaie de redemarer HA en ayant cocher et decocher "generate energy meter" plusieurs fois. Le probleme ne survient qu'au demarrage, il faut donc vraiment forcer le redemarrage de HA.

ic-dev21 commented 6 months ago

En test pour le week-end. @elafontaine as-tu des tests particuliers que tu voudrais que je fasse? Je viens de le pousser sur un clean install en dev, avec seulement "generate energy meters" coché. Autre chose?

Essaie de redemarer HA en ayant cocher et decocher "generate energy meter" plusieurs fois. Le probleme ne survient qu'au demarrage, il faut donc vraiment forcer le redemarrage de HA.

Alright pas de trouble. Ça te dérange pas si j'embarque dans le train de ton PR pour régler quelques autre bricoles?

ic-dev21 commented 6 months ago

Je vais inclure ici un fix pour #394

elafontaine commented 6 months ago

CA ne me derange pas, mais je vais clairement over-writer tes affaires avec mon workflow XD.

honnetement, je prefererais qu'on ne se pile pas sur les pieds si possible.

From: Ian C. @.> Sent: 15 March 2024 20:58 To: dvd-dev/hilo @.> Cc: Eric Lafontaine @.>; Mention @.> Subject: Re: [dvd-dev/hilo] Handle utility_meter Tariffs list to always exist (PR #392)

Je vais inclure ici un fix pour #394https://github.com/dvd-dev/hilo/issues/394

— Reply to this email directly, view it on GitHubhttps://github.com/dvd-dev/hilo/pull/392#issuecomment-2000871947, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAXCYVA7ND7PPEKMA5P35GLYYOKMLAVCNFSM6AAAAABEYTY7CWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBQHA3TCOJUG4. You are receiving this because you were mentioned.Message ID: @.***>

ic-dev21 commented 6 months ago

CA ne me derange pas, mais je vais clairement over-writer tes affaires avec mon workflow XD. honnetement, je prefererais qu'on ne se pile pas sur les pieds si possible. … ____ From: Ian C. @.> Sent: 15 March 2024 20:58 To: dvd-dev/hilo @.> Cc: Eric Lafontaine @.>; Mention @.> Subject: Re: [dvd-dev/hilo] Handle utility_meter Tariffs list to always exist (PR #392) Je vais inclure ici un fix pour #394<#394> — Reply to this email directly, view it on GitHub<#392 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAXCYVA7ND7PPEKMA5P35GLYYOKMLAVCNFSM6AAAAABEYTY7CWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBQHA3TCOJUG4. You are receiving this because you were mentioned.Message ID: @.***>

Je préfères éviter mais je ne peux pas m'auto-approuver, alors si je piggy-back ça va me permettre de sortir un fix plus vite.

Gènes-toi pas de venir faire un tit tour sur Discord https://discord.gg/bGfscCgr pour en jaser, tu es le bienvenue!

elafontaine commented 6 months ago

Je ne reussi pas a rejoindre le discord que tu m'as partager... une erreur sur l'invite on dirait...

Regards, Éric Lafontaine

On Fri, Mar 15, 2024 at 9:15 PM Ian C. @.***> wrote:

Merged #392 https://github.com/dvd-dev/hilo/pull/392 into main.

— Reply to this email directly, view it on GitHub https://github.com/dvd-dev/hilo/pull/392#event-12138023462, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAXCYVBNVPIXWPDFYKA5AWLYYOMLTAVCNFSM6AAAAABEYTY7CWVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGEZTQMBSGM2DMMQ . You are receiving this because you were mentioned.Message ID: @.***>

elafontaine commented 6 months ago

Aussi, j'ai foirer XD j'ai fait une erreur sur le niveau de la cle dans le dictionnaire. Regards, Éric Lafontaine

On Fri, Mar 15, 2024 at 9:18 PM Eric Lafontaine @.***> wrote:

Je ne reussi pas a rejoindre le discord que tu m'as partager... une erreur sur l'invite on dirait...

Regards, Éric Lafontaine

On Fri, Mar 15, 2024 at 9:15 PM Ian C. @.***> wrote:

Merged #392 https://github.com/dvd-dev/hilo/pull/392 into main.

— Reply to this email directly, view it on GitHub https://github.com/dvd-dev/hilo/pull/392#event-12138023462, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAXCYVBNVPIXWPDFYKA5AWLYYOMLTAVCNFSM6AAAAABEYTY7CWVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGEZTQMBSGM2DMMQ . You are receiving this because you were mentioned.Message ID: @.***>

ic-dev21 commented 6 months ago

Je ne reussi pas a rejoindre le discord que tu m'as partager... une erreur sur l'invite on dirait... Regards, Éric Lafontaine … On Fri, Mar 15, 2024 at 9:15 PM Ian C. @.> wrote: Merged #392 <#392> into main. — Reply to this email directly, view it on GitHub <#392 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAXCYVBNVPIXWPDFYKA5AWLYYOMLTAVCNFSM6AAAAABEYTY7CWVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGEZTQMBSGM2DMMQ . You are receiving this because you were mentioned.Message ID: @.>

https://discord.gg/VpEqhJdz

Est-ce que celui-là marche?