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

fix: The progress bar in Datatables of both Clients and Groups loads endlessly. #629

Merged
merged 1 commit into from
Jan 11, 2019

Conversation

AbhilashG97
Copy link
Member

@AbhilashG97 AbhilashG97 commented Mar 21, 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.

Issue Reference :
#597

After fixing the issue-
The progress bar now doesn't load endlessly on going back from a Datatable to the list of Datatables.

videotogif_2017 03 21_18 24 19

@AbhilashG97
Copy link
Member Author

AbhilashG97 commented Mar 21, 2017

I have made the required changes, please review my pull request. And kindly let me know if I have to make any further changes.

@therajanmaurya
Copy link
Member

@AbhilashG97 Can you please remove this line ((MifosBaseActivity) getActivity()).getActivityComponent().inject(this); to public View onCreateView() from public void onCreate() and Let me know what happen.

@AbhilashG97
Copy link
Member Author

@therajanmaurya, I made the suggested change, and there seems to be no difference in the behavior of the datatables, the endless loading is still not present, as expected. I am attaching the GIF of the application after I made the suggested change. The logs for the same can be found here. Also, should I apply these changes in the pull request ? Kindly let me know.

videotogif_2017 03 28_18 40 11

@therajanmaurya
Copy link
Member

Do one more thing, instead of initializing the subscriptions = new CompositeSubscription(); in attachView, remove subscriptions.unsubscribe(); from detachView and add subscriptions.clear();.
Let me know.

@AbhilashG97
Copy link
Member Author

@therajanmaurya, I have made the suggested changes, and the progress bar doesn't load endlessly. Please let me know if I have to make any further changes. GIF for the same can be found below.

GIF-

videotogif_2017 04 24_18 43 35

@tarun0 tarun0 merged commit c143811 into openMF:master Jan 11, 2019
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.

3 participants