Skip to content

Conversation

@Didayolo
Copy link
Member

@Didayolo Didayolo commented Apr 12, 2025

Description

Update public benchmarks list:

  • Add filter Has reward
  • Add sorting options: Most recent, Participants Count, Submissions Count
Capture d’écran 2025-04-12 à 13 27 54 Capture d’écran 2025-04-12 à 13 28 01

In progress

It is a first implementation of the logic, which is already an enhancement.
However, we can improve the UI - currently it's not pretty.

Issues this PR resolves

A checklist for hand testing

  • Go to public listing
  • Play with pagination and filters
  • Check that the search bar and other listings are not broken

Checklist

  • Code review by me
  • Hand tested by me
  • I'm proud of my work
  • Code review by reviewer
  • Hand tested by reviewer
  • CircleCi tests are passing
  • Ready to merge

@ObadaS
Copy link
Collaborator

ObadaS commented Apr 29, 2025

I tested this on the production database. It is pretty quick when I am the only using it. However, if I try to launch something else that uses the Database, it slows down the feature.
Maybe it can be optimized a bit more to not do a lot of calls to the Database each time someone tries to use the feature.

@Didayolo
Copy link
Member Author

@ihsaan-ullah If you wish to work on the public list filters, you can see this PR as a basis to help you

@Didayolo
Copy link
Member Author

We close this PR and focus on:

@Didayolo Didayolo closed this Jun 25, 2025
@Didayolo Didayolo deleted the listing-filters branch June 25, 2025 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants