Thank you for the detailed background and context, Anton. I agree that using
date pickers makes sense if we’re filtering notes by dates, especially since
the current setup isn’t relying on standard pagination.
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 first and then refactor this PR accordingly.
On a side note, I’ve managed to implement sortable columns for `created_at` and
`updated_at` locally, but I’ll hold off on submitting that until this PR is
merged to keep things cleaner and easier to review.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5255#issuecomment-2417243495
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/5255/c2417243...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
