[openstreetmap/openstreetmap-website] Add sorting and filtering functionality to user notes page (PR #5255)

Anton Khorev notifications at github.com
Thu Oct 17 10:12:18 UTC 2024


> As for the next steps, I’m not sure whether it’s better to get this PR merged as is and refactor later when the notes move to standard pagination, or if we should address the pagination issue on user note pages first and then refactor this PR accordingly.

You can make a pull request that just adds the status filter. That doesn't depend on pagination and shouldn't tank the performance because:
- status is checked anyway to see if the note is not hidden
- we're doing a similar thing in the api

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5255#issuecomment-2419119591
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/5255/c2419119591 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20241017/57c46e0e/attachment.htm>


More information about the rails-dev mailing list