BottlecapDave / HomeAssistant-OctopusEnergy

Unofficial Home Assistant integration for interacting with Octopus Energy
https://bottlecapdave.github.io/HomeAssistant-OctopusEnergy/
MIT License
581 stars 60 forks source link

Current energy cost sensor displays $ symbol instead of £ #47

Closed ecylcje closed 2 years ago

ecylcje commented 2 years ago

Hi,

My current energy cost sensor icon is displayed as a $ instead of £ symbol.

Dollar_HA

sensor.octopus_energyelectricity{{METER_SERIALNUMBER}}{{MPAN_NUMBER}}_current_rate

Is this my setup or an issue because of:

HomeAssistant-OctopusEnergy-4.1.3\custom_components\octopus_energy\sensor.py (5 hits) Line 196: return "mdi:currency-usd" Line 274: return "mdi:currency-usd" Line 447: return "mdi:currency-usd" Line 531: return "mdi:currency-usd" Line 712: return "mdi:currency-usd"

I have configured everything else in Homeassistant to have GBP £ and it works fine. Until I upgraded octopus_energy recently (today) it was working there as well.

Thanks,

Chris

townsmcp commented 2 years ago

@ecylcje have you gone into 'Configuration' -> 'Settings' -> 'General' and set Currency to GBP?

ecylcje commented 2 years ago

Sadly yes. I checked in there and it is set to GBP:

Pound_HA

It was working fine before I upgraded octopus_energy versions. My old sensor was just sensor.octopus_energy_electricity_current_rate before it moved to the version with MPAN in. That was displayed nicely with £.

Thanks,

Chris

BottlecapDave commented 2 years ago

Hi, sorry that you are having issues. In your image, are you referrring to the $ icon as it looks like it's reporting in GBP? If not, what version of home assistant are you running?

Thanks, Dave

ecylcje commented 2 years ago

Thanks Dave.

It is just the icon image ($) which I was talking about. On my previous version (of Octopus-energy) it was a £ symbol.

I'm running the latest release of HA (22.03.08) on RPi. Obviously this is in no way critical or a major problem. It was just more a 'weird' issue I thought I would raise.

Thanks so much,

Chris

airampg commented 2 years ago

Hi people. I just wanted to let you know that I can also see this problem, with the currency set to GBP and so on. I just changed myself to be mdi:currency-gbp, so no biggie, but if it could be quickly fixed on the code it would also be nice for future updates :)

BottlecapDave commented 2 years ago

I have updated this in the latest release (v4.2.0)