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: sync for centers #724

Merged
merged 1 commit into from
Aug 7, 2017
Merged

Conversation

mayank-kgp
Copy link
Contributor

@mayank-kgp mayank-kgp commented Jul 28, 2017

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Apply the MifosStyle.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.
    Add sync for centers #716

@mayank-kgp
Copy link
Contributor Author

@therajanmaurya I have changed some code related to center sync. Now it is working fine.
You can also test.

@therajanmaurya
Copy link
Member

@mayank-kgp without an internet connection, I can see the center details, group list and not even the center clients. because you sync the data but you did not make any calls to the database if the user is in offline mode.

This issue is not resolved yet, if the user can't see the sync data then It's not making any sense.

@therajanmaurya
Copy link
Member

I am merging this PR now, send another PR for showing the database center details, group list, clients.

@therajanmaurya therajanmaurya merged commit 9b5c639 into openMF:master Aug 7, 2017
mayank-kgp pushed a commit to mayank-kgp/android-client that referenced this pull request Aug 8, 2017
mayank-kgp pushed a commit to mayank-kgp/android-client that referenced this pull request Aug 8, 2017
mayank-kgp pushed a commit to mayank-kgp/android-client that referenced this pull request Aug 9, 2017
mayank-kgp pushed a commit to mayank-kgp/android-client that referenced this pull request Aug 9, 2017
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.

2 participants