-
Notifications
You must be signed in to change notification settings - Fork 3
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 API responeses #13
Comments
The old data that was retrieved should be saved in the app configuration folder. It's not very likely that a company like the data provider would provide historical misleading data, so for the old data it should be cached until the user wants to wipe out the cache and refresh all the data. If there is a expiration date of the cached data, it should not be too soon or too far. This is specially true because of this:
|
I agree so far with all your points and will keep this in mind when it cones to implementation. |
Searches for the same query shouldn't send over and over again to the API in a certain period of time.
The results could be stored for some minutes before sending again to the API.
Dont query frequently on weekends.
Don't forget to clear the list if it's getting too old.
The text was updated successfully, but these errors were encountered: