search

FaserF / ha-rewe

Rewe Discounts Homeassistant Integration
GNU General Public License v3.0
15 stars 4 forks source link

Some data is missing #6

Open moerpel opened 1 year ago

moerpel commented 1 year ago

Hi, The api delivers some more data i.e. subtitle. You somehow skip it in the component. I have already tried to include it by uncommenting but they are not in once I reload the component.

Would you be able to integrate such too...?

dawnbreak commented 1 year ago

I had the same thought, subtitle is needed. I got confused when I had 5 times the same product in my list and the information to distinguish it, was in the subtitle. But all the code is already there and just had to uncomment it. I think it only updates once a day, so it takes a bit time. I also needed the NAN product ID for other integrations and added it. It also works, but there are some problems adding all these additional values: WARNING (Recorder) [homeassistant.components.recorder.db_schema] State attributes for sensor.rewe_XXX exceed maximum size of 16384 bytes. This can cause database performance issues; Attributes will not be stored I am not quite sure what is the best way forward. I thought about adding a configuration that I only return the categories I need, or maybe it would be better to have one sensor per category? I have not much experience with HA yet, so I am not quite sure what the best way would be.