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

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


> Sure, but as I understand it this was an urgent change to assist DWG with their efforts to fight the ongoing vandalism which is why I was keen to merge and deploy it as quickly as possible - obviously we had not anticipated that people would rely on either the ordering or the 404 behaviour.

I understand your point. Identifying non-obvious breaking changes can be challenging. However, the manner in which you address such issues is crucial. Making assertive statements like the ones, https://github.com/openstreetmap/openstreetmap-website/commit/b9c85c269726faad3c2613b37f01683c8b59f5c6#commitcomment-125593592, https://github.com/openstreetmap/openstreetmap-website/commit/be96aa7cb42a2d43f84717a023f57f52b38cf6b7#commitcomment-125593392, is, in my opinion, not the most effective approach. A more constructive method of handling such situation would be welcome.

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

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


More information about the rails-dev mailing list