myTselection / Carbu_com

Actual fuel prices (super, diesel, lpg) per gasstation supported for Belgium, France, Luxembourg, Spain, Netherlands, Germany, Italy and US! Home Assistant custom component HACS integration for BE/FR/LU based on carbu.com and mazout.com site to compare and save on your actual fuel oil/heating oil (mazout) and fuel (diesel, super and lpg) purchases.
MIT License
23 stars 6 forks source link

Station images corrupt? #28

Closed LJStevo closed 1 year ago

LJStevo commented 1 year ago

(sorry for my amateur input) Station image url is not what it should be? entity_picture: https://www.prezzibenzina.it/www2/marker.php?brand=GA&status=AP&price=1.929&certified=0&marker_type=3

myTselection commented 1 year ago

Not all stations are known by this site. I'm (mis)using their icons to mostly get better icon's on the map with price indication. So I won't be able to fix this.

LJStevo commented 1 year ago

Strange, cause they worked before? They are all here? https://carbu.com/belgie//tankstations

myTselection commented 1 year ago

They do still work, don't they? Only for some brands the icon may be missing but price is still shown. Or can you give more details on your issue?

LJStevo commented 1 year ago

Sorry for late reaction. I think it's for all stations, for me atm "Maes" and "Avia" Here a picture, also the fault in postal code 9100 showing Zwijndrecht instead of Sint-Niklaas afbeelding afbeelding

myTselection commented 1 year ago

I had changed the logo's, no longer using the logo's by carbu, but using instead the logo's of italian site. The benefit of the new logo's is that they can also immediately show the price, which i found usefull (eg on the map). I might add an option in the config flow to allow you the select if the italian or carbu logo should be used

myTselection commented 1 year ago

Fixed in R7.3.0 with extra option during sensor setup