jrfernandes / ontario_energy_board

Home Assistant component that installs a sensor with the current energy rate for Ontario energy companies
MIT License
41 stars 10 forks source link

Invalid currency code : CA$/kWh #6

Closed McGeaverBeaver closed 2 years ago

McGeaverBeaver commented 2 years ago

Sensor is reporting incorrect currency code as per Home Assistant.

RangeError: Invalid currency code : CA$/kWh

Code should be CAD/kWh

Thank you.

jrfernandes commented 2 years ago

I'm curious, where/how did you see that error?

McGeaverBeaver commented 2 years ago

This was in the home assistant logs from the UI. I would see it after visiting a dashboard with the entity. It's no longer being displayed in the logs, if I recreate I will post more details. Sorry I left them out the first time.

On Thu., Mar. 3, 2022, 23:01 Joel Fernandes, @.***> wrote:

I'm curious, where/how did you see that error?

— Reply to this email directly, view it on GitHub https://github.com/jrfernandes/ontario_energy_board/issues/6#issuecomment-1058804743, or unsubscribe https://github.com/notifications/unsubscribe-auth/AG7T7YIHEPOO2MFXVMFNH53U6GDLLANCNFSM5PGB5I2Q . You are receiving this because you authored the thread.Message ID: @.***>

kerhbal commented 2 years ago

something is wrong with the unit, not sure I have the same problem, here is a screenshot:

Safari-Settings – Home Assistant--2022-05-10 at 10 59 25
McGeaverBeaver commented 2 years ago

Right! Yes, That's where it was from!

On Tue., May 10, 2022, 11:00 kerhbal, @.***> wrote:

something is wrong with the unit, not sure I have the same problem, here is a screenshot: [image: Safari-Settings – Home Assistant--2022-05-10 at 10 59 25] https://user-images.githubusercontent.com/11629684/167659585-d33954f9-8f33-48e0-96bd-5154f48233e2.png

— Reply to this email directly, view it on GitHub https://github.com/jrfernandes/ontario_energy_board/issues/6#issuecomment-1122511666, or unsubscribe https://github.com/notifications/unsubscribe-auth/AG7T7YMEXL5XY742F2UJ2Q3VJJ2YJANCNFSM5PGB5I2Q . You are receiving this because you authored the thread.Message ID: @.***>

jrfernandes commented 2 years ago

That's odd. What currency do y'all have set under Settings > System > General (/config/general)?

Is it CAD?

Screen Shot 2022-05-13 at 5 19 23 PM

kerhbal commented 2 years ago

@jrfernandes yep

Safari-Settings – Home Assistant--2022-05-13 at 17 25 16
kerhbal commented 2 years ago

though I found that the value works in energy dashboard with currency, just something is weird with that error message(I still can see it despite currency works)

Safari-Energy – Home Assistant--2022-05-13 at 17 26 40