rox icon indicating copy to clipboard operation
rox copied to clipboard

Bug with gender search filter

Open ghost opened this issue 4 years ago • 4 comments

Bug reported by Polyglot, but not solved yet so I post it here :

  • The search for a host with gender "other" returns all hosts instead.
  • Searches for male and searches for females hosts seem to work on page 1, but display all hosts as soon as I click beyond the first page.

ghost avatar Mar 06 '21 15:03 ghost

Second part is still open.

thisismeonmounteverest avatar Mar 07 '21 09:03 thisismeonmounteverest

@thisismeonmounteverest it is the other way around. "other" still does not work.

tillwenke avatar Jul 27 '22 20:07 tillwenke

The search for a host with gender "other" returns all hosts instead.

Planning to investigate this as a first issue. Feel free to assign the issue to me, if possible @ghost

nciemniak avatar Sep 12 '22 15:09 nciemniak

The search for a host with gender "other" returns all hosts instead.

Planning to investigate this as a first issue. Feel free to assign the issue to me, if possible @ghost

I am also just starting. Tried to find out more about the structure of the project here https://github.com/BeWelcome/rox/issues/296#issue-1320094636 hope this helps us at least a bit to get started

tillwenke avatar Sep 12 '22 18:09 tillwenke

It appears to work properly when I test it locally, i.e. searching for hosts with gender "other" in fact returns only hosts with gender "other." Makes me wonder if it could be a data issue? Has anyone been able to replicate this locally? (maybe @TillWenke, @thisismeonmounteverest ?)

My next route of action would normally be to take a look at the data in the production database (or read replica, if exists), logging in with read-only permissions for safety, but considering I am new, I am reluctant to ask this sort of admin-exclusive access.

nciemniak avatar Sep 27 '22 12:09 nciemniak

@nciemniak Seems I fixed that already :)

Regarding access to production data that would need an NDA but trying it on beta should be enough if your local code is close to the deployed code there.

thisismeonmounteverest avatar Sep 27 '22 18:09 thisismeonmounteverest

@thisismeonmounteverest Nice 👍 How do I access beta?

Also, I'm still seeing the issue on production, so is the fix only pushed to beta?

nciemniak avatar Sep 27 '22 20:09 nciemniak