[openstreetmap/openstreetmap-website] Restore original users index method; add search method for multi fetch (PR #4207)

Tom Hughes notifications at github.com
Mon Aug 28 18:01:18 UTC 2023


Yes I know - that was my point that by adding search as a parallel to index (but without the 404) you can then revert index to the 404 behaviour, but I see no reason to want the old behaviour.

I hadn't looked into the history of it so maybe accident isn't the right word but it's clearly not a useful design so I have no idea why I did it like that.

My guess is that I based it on the single user fetch and didn't really think it through - before last week I wasn't even really aware that `find` could take multiple IDs as I would normally use `where` where selecting more than one object so maybe I literally copied the code and it worked and I didn't really notice the silliness.

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

Message ID: <openstreetmap/openstreetmap-website/pull/4207/c1696114330 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20230828/58c7c2da/attachment.htm>


More information about the rails-dev mailing list