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

"Load More" in People tab does not apply filter to returned results #2143

Closed
lriggle-strib opened this issue Jan 4, 2019 · 7 comments
Closed
Labels

Comments

@lriggle-strib
Copy link

lriggle-strib commented Jan 4, 2019

Do you want to request a feature or report a bug?

Bug

Intended outcome:

After filtering users by role, clicking "Load More" should display more users of that role

Actual outcome:

After filtering users by role, clicking "Load More" does not display more users of that role, it displays users of any role.

How to reproduce the issue:

  1. Filtered all people so only Administrators would show.
    • There are currently 14 administrators in the database.
    • Initial pageload displays 10 users
  2. Click "Load More"
  3. The additional 4 administrators should display. Instead 5 additional users of any role display.

screen shot 2019-01-04 at 12 03 31 pm

Version and environment

We're currently on 4.6.5. We had some issues upgrading to newer versions prior to our launches last month and haven't circled back around to figure out what went wrong.

@immber immber added the bug label Jan 4, 2019
@immber
Copy link
Contributor

immber commented Jan 4, 2019

This looks related to a similar bug ticket we have open for an issue with the Assets (Stories) page pagination. I've added a note to that bug ticket to investigate this at the same time. https://www.pivotaltracker.com/story/show/162740247

@immber
Copy link
Contributor

immber commented Jan 29, 2019

May be related to #2158, which is also reported as being and issue with Assets, but could be hitting the same issue of query missing limit.

@kgardnr
Copy link
Contributor

kgardnr commented Feb 5, 2019

Fixed in #2174 @lriggle-strib!

@kgardnr kgardnr closed this as completed Feb 5, 2019
@kgardnr kgardnr reopened this Feb 6, 2019
@lriggle-strib
Copy link
Author

Thanks @kgardnr!

@mkonikov
Copy link

Hey @immber -

I'm still finding this to be an issue in v4.8.0

@immber
Copy link
Contributor

immber commented Mar 14, 2019

@kgardnr You closed this as fixed in #2174, however that PR only addressed the issue on the Stories view, I believe this is still an issue on the People tab, so I'm re-opening. We might need to apply the same or similar fixes to Users as were done to Assets.

@immber immber reopened this Mar 14, 2019
@kgardnr
Copy link
Contributor

kgardnr commented Mar 14, 2019

Thanks @immber!

@mkonikov and/or @lriggle-strib would you mind submitting a fix for this potentially? Our team is heads down on v5 the next few weeks so it would really help us out if you could!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants