[openstreetmap/openstreetmap-website] Use where instead of find to prevent 404 (b9c85c2)

Kamil Monicz notifications at github.com
Sat Aug 26 15:16:53 UTC 2023


> Did you expect 404 when one user was missing or when all of them were?

Previous logic worked this way:
* Return 404 if at least one user is missing

While a very weird behavior indeed, I wrote my code around this issue. I have seen worse things in my life.

The current logic works this way:
* Always return 200 and skip not found users

My previously coded applications are not ready for such change.

The green lines are additions I had to do today when I noticed my app stopped working:

![image](https://github.com/openstreetmap/openstreetmap-website/assets/10835147/6fa77113-cdb7-4ee9-870f-7a3f23133ecc)

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/commit/b9c85c269726faad3c2613b37f01683c8b59f5c6#commitcomment-125594991
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/commit/b9c85c269726faad3c2613b37f01683c8b59f5c6/125594991 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20230826/3c55c90b/attachment.htm>


More information about the rails-dev mailing list