[openstreetmap/openstreetmap-website] Support versions in elements multi fetch (PR #3715)

Anton Khorev notifications at github.com
Fri Jun 27 15:07:23 UTC 2025


AntonKhorev left a comment (openstreetmap/openstreetmap-website#3715)

I decided to address this completely:

> Possibly it would better to add new tests rather than add lots of cases to the existing ones. I suspect @gravitystorm will think so though he'd probably also say we should split the existing one.

and split all `index` action tests, exisitng and added in this PR. Then I discovered that I never checked if old versions of elements were redacted. Added a fix for that too.

The expected behavior if one of the requested versions is redacted wasn't documented anywhere, so I copied what cgimap did, which is 404 like for missing versions and missing elements.

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

Message ID: <openstreetmap/openstreetmap-website/pull/3715/c3013398792 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250627/dc1158e5/attachment.htm>


More information about the rails-dev mailing list