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

feat:offline support in the home fragment wrt #1122 #1938

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

PratyushSingh07
Copy link
Collaborator

Fixes #1122

Before the Change:
https://user-images.githubusercontent.com/90026952/223648417-cabed712-84ce-4d88-aa56-5b2e7f06d48b.mp4

After the Change:

WhatsApp.Video.2023-03-08.at.12.51.26.mp4
  • Apply the AndroidStyle.xml style template to your code in Android Studio.

  • Run the unit tests with ./gradlew check to make sure you didn't break anything

  • If you have multiple commits please combine them into one commit by squashing them.

@PratyushSingh07
Copy link
Collaborator Author

@jawidMuhammadi your thoughts on this feature? As the issue had mentioned this can be the first step in providing a better offline support to the app

@jawidMuhammadi
Copy link
Contributor

  • This PR includes codes not related to what is written in its title. It would be best to create new branches for each issue from the origin/openMF/development branch and also each PR should contain the changes/solution for the issue which has been linked not more than that.

  • No, I don't think saving data objects in the shared preferences is a good idea as it is described in the description of the issue, because shared preferences are intended to be used to save small data in key-value formats. To support a proper offline first approach we would need to use a database that is out of scope at the moment.

@PratyushSingh07 PratyushSingh07 force-pushed the optional-support branch 2 times, most recently from a5406c8 to d8c3220 Compare March 9, 2023 12:26
@PratyushSingh07
Copy link
Collaborator Author

@jawidMuhammadi fixed the PR

@PratyushSingh07 PratyushSingh07 changed the title offline support in the home fragment wrt #1122 feat:offline support in the home fragment wrt #1122 Mar 23, 2023
@PratyushSingh07
Copy link
Collaborator Author

its been long since we last interacted on this PR @jawidMuhammadi . Do you think that we can add this feature for the time being in our project ?

Copy link
Collaborator

@luckyman20 luckyman20 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

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.

Offline support for the home screen.
3 participants