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

After refreshing any tab in Requests Page, the respective tab should be displayed #1110

Open
CodeChamp-SS opened this issue Mar 7, 2021 · 13 comments
Assignees
Labels
Category: Coding Changes to code base or refactored code that doesn't fix a bug. First Timers Only Good for newcomers.

Comments

@CodeChamp-SS
Copy link
Contributor

Describe the bug

On refreshing through either past or all tab in the requests page, the user is redirected to the pending tab (as seen in the video), which shall not happen.

To Reproduce

Steps to reproduce the behavior:

  1. Go to 'Requests page'
  2. Swipe down to refresh either PAST or ALL tab
  3. See the problem

Expected behavior

The same tab should be displayed after refreshing too.

Video

2021-03-07.15-02-23.mp4
@CodeChamp-SS
Copy link
Contributor Author

I would like to take up this issue if it's approved.

@epicadk epicadk added Category: Coding Changes to code base or refactored code that doesn't fix a bug. First Timers Only Good for newcomers. labels Mar 7, 2021
@ManishPanwar556
Copy link
Contributor

I would like to work on this issue @epicadk

@epicadk
Copy link
Member

epicadk commented Apr 11, 2021

Sorry, @ManishPanwar556 it's a first-timers-only issue and you have already contributed to the repository before.

@Marvel999
Copy link
Contributor

I would like to work on this issue @epicadk

@Marvel999 Marvel999 mentioned this issue Apr 20, 2021
4 tasks
@epicadk
Copy link
Member

epicadk commented Apr 20, 2021

I would like to work on this issue @epicadk

Hi sorry, this is a first timers only issue and you have already contributed to this repository before. Please find another issue to work on.

@Marvel999
Copy link
Contributor

I would like to work on this issue @epicadk

Hi sorry, this is a first timers only issue and you have already contributed to this repository before. Please find another issue to work on.

ok

@shivamparihar12
Copy link

Hi, I would like to work on this issue @epicadk

@epicadk
Copy link
Member

epicadk commented May 8, 2021

Hi, I would like to work on this issue @epicadk

assigned : ).

@shivamparihar12
Copy link

I am going to be honest, after trying for 2 days I still don't know how to rectify this issue. Is there anyone who can help me?

@epicadk
Copy link
Member

epicadk commented May 10, 2021

I am going to be honest, after trying for 2 days I still don't know how to rectify this issue. Is there anyone who can help me?

Have you been able to replicate it?

@shivamparihar12
Copy link

shivamparihar12 commented May 11, 2021

I am going to be honest, after trying for 2 days I still don't know how to rectify this issue. Is there anyone who can help me?

Have you been able to replicate it?

yes

@isabelcosta
Copy link
Member

@shivamparihar12 Are you still interested in working on this? I don't have an idea of how to fix this, but someone in the community could have. If you're still interested do ask on Zulip or let us know what you found out so far.

Do you know what is causing the bug? Which parts of the code are causing this?

@shivamparihar12
Copy link

Yes @isabelcosta , I would love to solve this. So, I could be of some use to the community.

These screenshots are from the RequestsFragemnt.kt.

I could be wrong here but according to me this function fetchNewest() should call srlRequest.isrefreshing = false at the end
Screenshot (705)

and when the user opts to refresh the viewpager2 using menu item instead of the swipe gesture. We have to set srlRequest.isrefreshing = true before calling fetchNewest()
Screenshot (706)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Coding Changes to code base or refactored code that doesn't fix a bug. First Timers Only Good for newcomers.
Projects
None yet
Development

No branches or pull requests

6 participants