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

Day description in the weekly weather forecast is wrong #162

Open
Joao31415 opened this issue Oct 7, 2024 · 5 comments
Open

Day description in the weekly weather forecast is wrong #162

Joao31415 opened this issue Oct 7, 2024 · 5 comments

Comments

@Joao31415
Copy link

Steps to reproduce:

  1. Go to Weekly forecast
  2. Temperature data and icon seems to be right but day description seems to be one day behind.

Example:
Considering that the icon and temperature are right, when you read today should be tomorrow and so on for the rest of the list.
In other words, considering that today is Sunday, when you read Today, should be Tomorrow. When you read Tomorrow, should be Tuesday. When you read Tuesday, should be Wednesday and so on.

This bug happens in Brazilian Portuguese. I do not know if it happens in other languages too.

Screenshot_20241006_235703

@amit9838
Copy link
Owner

Ok so you mean app should not display today row in weekly forecast?

@Joao31415
Copy link
Author

Joao31415 commented Oct 13, 2024

Ok so you mean app should not display today row in weekly forecast?

No. That's not the case.
There is a bug in the application that the icon and temperature displayed in today's row represents the forecast for tomorrow when the expected in Today's row would be the Icon and the Min/Max temperature for Today.
The same happens in all the other rows.

@amit9838
Copy link
Owner

amit9838 commented Oct 21, 2024

Hi @Joao31415 , I cross verified the app with the data, i didn't see any issue in my case.
Here are screenshots.

Screenshot 2024-10-21 at 10 51 35 AM Screenshot 2024-10-21 at 10 51 52 AM

To make sure everything is working as expected, I'll need your help in this.

This is the api endpoint, pls visit link and enter your desired location, also make sure to select same location in the app. Take a screenshot of both app and api response with 7day forecast. (pls Make sure to use English lang in the app so that i can understand 😄 )
https://open-meteo.com/en/docs#daily=weather_code,temperature_2m_max,temperature_2m_min&timezone=auto

@Joao31415
Copy link
Author

Joao31415 commented Oct 21, 2024

I collected the data today: Monday, October 21st.
I translated the days as displayed by Mousam to English and added the date corresponding to the day of the week.

231021 API Mousam

Update: Perhaps, this issue is happening because here is (GMT-3)

@amit9838
Copy link
Owner

Oh, got it, y're right it I'm very sure It's timezone issue. Will look into it. BTW thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants