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

Cache last valid response and add 'last updated at' info #33

Open
YayC opened this issue Apr 4, 2016 · 3 comments
Open

Cache last valid response and add 'last updated at' info #33

YayC opened this issue Apr 4, 2016 · 3 comments

Comments

@YayC
Copy link

YayC commented Apr 4, 2016

Great widget! I've just been having an issue where the "Could not retrieve weather data" error is popping up quite frequently, which wipes out the whole forecast from the view. You can see how it looks if you turn off your internet connection when the widget tries to refresh.

When there is an error fetching new data, it would be awesome if it would keep data from the last valid response since that (slightly outdated) info is still very useful. A complementary feature would be to display a "last updated at" field so the user would know how fresh the info is.

@felixhageloh
Copy link
Owner

good suggestion!

@wezzer
Copy link

wezzer commented Jun 12, 2016

Thumbs up for this! Excellent widget, but requires live connection to Internet all the time...

@kyletitus
Copy link
Contributor

I have put in a pull request that should resolve this issue. It adds caching and some retry logic. You can review it/test it out here. #37

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

No branches or pull requests

4 participants