search

briis / affalddk

Home Assistant Integration to retrieve Garbage Collection data from danish Municipalities
MIT License
21 stars 5 forks source link

"Genbrug" vises forkert på adressen #67

Closed frankebobdk closed 5 months ago

frankebobdk commented 5 months ago

Describe the issue

Hej,

Først og fremmest vil jeg gerne takke dig for den indsats, du lægger i denne udvidelse!

Jeg har bemærket, at på min adresse og generelt for Mariagerfjord Kommune, vises "Genbrug" ikke korrekt som den affaldsbeholder, den faktisk er. For at præcisere, så dækker genbrugsaffaldet faktisk over "papir/pap/metal/plast/mad- og drikkekartoner".

Kan du (såfremt muligt) ændre det, så "Genbrug" vises som "papirglasmetalplast.svg"? Jeg mener, at dette vil være den mest retvisende angivelse.

What version of Home Assistant Core has the issue?

core-2024.4.0

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

What version of the integration are you using?

2.1.1

Diagnostics information

Material type [240 l genbrug låg i låg (1 stk.)] is not defined in the system for Genbrug. Please notify the developer.

Anything in the logs that might be useful for us?

No response

Additional information

No response

briis commented 5 months ago

Det kan jeg sagtens. Jeg har lige testet med en adresse i Hobro, og her var der ganske rigtigt en container der hed 240 l genbrug låg i låg (1 stk.). Men for at være helt sikker på det er samme container hos dig, kunne du sende mig din adresse på bjarne@briis.com, så tester jeg lige at det virker. Ved du allerede at den hedder det samme, så er alt OK, og den er lagt i papir/pap/metal/plast/mad- og drikkekartoner og du behøver ikke sende noget.

frankebobdk commented 5 months ago

Adresse sendt på mail og tak for hjælpen 👍

frankebobdk commented 5 months ago

Ved ikke om det skal noteres her, men næste afhentning viser stadig det gamle genbrug logo, og er ikke opdateret.

Har prøvet at reloade adressen, men viser stadig det gamle logo.

@briis

briis commented 5 months ago

Er det efter du har opdateret til 2.1.3? Den er lige frigivet for et par timer siden

frankebobdk commented 5 months ago

Ja har netop opdaget for cirka 20 min siden.

Se nedenstående billede

image

briis commented 5 months ago

Bare for at være helt sikker: og du har genstartet Home Assistent?

frankebobdk commented 5 months ago

Bare for at være helt sikker: og du har genstartet Home Assistent?

Ja har genstartet Home Assistant - 2 gange endda for at være sikker.

briis commented 5 months ago

Godt, så må jeg lige hvorfor det ikke virker. Beklager

briis commented 5 months ago

Det er sgu underligt. Jeg har lige lagt din adresse ind på en test udgave af Home Assistant, med V2.1.3 installeret. og den viser det helt som den skal:

Skærmbillede 2024-04-07 kl 15 54 05

Når jeg kigger nærmere på dit billede, så er logoet jo korrekt, det er kun teksten der vises forkert. Gad vide om det er et cache problem med browseren? Eller har du selv skrevet teksten?

frankebobdk commented 5 months ago

Tror vi snakker forbi hinanden 😊

Jeg mener logoet på entiteten "naeste afhentning". Burde den ikke have opdateret logoet til det korrekte?

briis commented 5 months ago

Ja 😀 vi snakker forbi hinanden. Nej det Logo er altid det samme. Grunden til det, er at der nogen, der får hentet flere containere på én dag, og så samler jeg teksten og beskrivelsen fra alle containere. Men det vil så bliver umuligt at lave alle logoer for de containere, da sammensætningen kan være rigtig mange forskellige. Derfor er næste afhentning altid et Genbrugs logo.

Denne ændring kom i V2.0.5 Se release notes her https://github.com/briis/affalddk/blob/main/CHANGELOG.md#version-205

frankebobdk commented 5 months ago

Beklager, det havde jeg ikke set 😊 Så virker alt som det skal. Du må gerne lukke dette issue.

Tak for hjælpen.