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: SwipeRefreshLayout spinning endlessly on the account section #217

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ksog66
Copy link

@ksog66 ksog66 commented Mar 5, 2021

Issue Fix

Fixes #216

Screenshots

WhatsApp.Video.2021-03-05.at.18.21.40.mp4

Description

Added code to stop the continuous spinning of SwipeRefreshLayout in account section and also added the action, it is supposed to do when refresh layout

  • 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.

@ksog66 ksog66 marked this pull request as ready for review March 5, 2021 13:45
@garvit984
Copy link
Collaborator

@ksog66 You have not followed the contribution guidelines. Please squash multiple commits into one.

@ksog66
Copy link
Author

ksog66 commented Mar 9, 2021

@garvit984 Actually i didnt know about that , i am new to open source. can you help with it as i dont know what is squasing commit ?

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.

SwipeRefreshLayout spin endlessly after refreshing in Accounts Fragment
2 participants