Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] weather-forecast dates are not correct #1376

Open
5 tasks done
mgryka opened this issue Nov 10, 2023 · 2 comments
Open
5 tasks done

[BUG] weather-forecast dates are not correct #1376

mgryka opened this issue Nov 10, 2023 · 2 comments
Assignees
Labels
🐛 Bug [ISSUE] Ticket describing something that isn't working

Comments

@mgryka
Copy link

mgryka commented Nov 10, 2023

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 mgryka added the 🐛 Bug [ISSUE] Ticket describing something that isn't working label Nov 10, 2023
@liss-bot
Copy link
Collaborator

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

@CrazyWolf13
Copy link
Collaborator

CrazyWolf13 commented Mar 3, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug [ISSUE] Ticket describing something that isn't working
Projects
Status: Up Next
Development

No branches or pull requests

4 participants