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

Update ForecastModule.java #16

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

FynnZW
Copy link

@FynnZW FynnZW commented Feb 29, 2016

My current temperatures are always off and I think I found the reason.
I'm in europe, so sending GMT minus 8 hours gives me the wrong data. According to the API, if no time is given the local time at coordinates is used, so I just took it out completely.

Please check carefully, I'm totally not sure and also don't know anything about android development :)

Found the real reason for my wrong current temps (I think).
I'm in europe, so sending GMT minus 8 hours gives me the absolute wrong data. According to the API, if no time is given the local time at coordinates is used, so I just took it out completely.

Please check carfeully, I'm totally not sure and also don't know anything about android development :)
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

Successfully merging this pull request may close these issues.

1 participant