dashy icon indicating copy to clipboard operation
dashy copied to clipboard

[BUG] weather-forecast dates are not correct

Open mgryka opened this issue 1 year ago • 2 comments

Environment

Self-Hosted (Docker)

System

Firefox 119.0

Version

2.1.1

Describe the problem

The weather information seems to be populating correctly, but the dates are not. See screenshot. image All dates except the last date are tomorrow's date. Last date is the day after tomorrow.

Also, I want to thank you for making Dashy! It's been a pleasure to work with so far.

Additional info

No response

Please tick the boxes

mgryka avatar Nov 10 '23 04:11 mgryka

If you're enjoying Dashy, consider dropping us a ⭐
🤖 I'm a bot, and this message was automated

liss-bot avatar Nov 10 '23 04:11 liss-bot

Just did a lot of testing and there seems to be a problem within the API.

I recreatet the way that dashy makes to get and display the data.

Made an api request to the same url as dashy and manually copied the date out, this date is provided in unix time, so dashy is using a converter. I used the same and even tested with another approch, look here for a one by one comparison.

https://jsfiddle.net/CrazyWolf/tgeayjus/ https://jsfiddle.net/CrazyWolf/ct3eL5p4/1/

While running the snippet, you can see that the time changes by 3 hours and not 24 as expected, I think we have to update the url or so, will look further into this later.

Digging further into this, it seems like we need to update the url to ..../forecast/daily... instead of /forecast

CrazyWolf13 avatar Mar 03 '24 20:03 CrazyWolf13