-
-
Notifications
You must be signed in to change notification settings - Fork 431
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
[Enhancement] Using local storage to reduce the server load. #438
Comments
@yasharth291 if you haven't already please read our contribution guidelines. Use issue and pr templates to create new issues and PR. |
Sorry for creating the issue but sir everyone takes the issues discussed in the slack and creates are here |
We are trying to give credit to everyone who is finding bugs/issues, suggesting new features, contributing any way to the project. If you felt someone else took credit for your hard work please reach out to us but not following guidelines is not good solution to this. Now, can you please update issue description ? |
yes sir sure i am updating it and can you please assign me to do this |
@Sagar2366 it is done |
thanks sir i am starting working on it now |
@yasharth291 As this is a feature. Apply the PR to the develop branch |
Sure Sir working on this now |
As i can see someone already has used and solved this problem using shared preferences. I am closing this issue |
Issue Description: The constant calling of api for details that are going to be used for longer time periods causes extra calls to api making the app slower so adding a local storage will help us.
Issue Severity: high
Environment Details: Issue found mostly in whole app
Observed Behaviour: The app is slow
Expected Behaviour: The app should be fast
Does this issue need immediate attention? yes
Are you willing to work on this issue: Yes, I am willing to work on this issue. Please assign this to me!
The text was updated successfully, but these errors were encountered: