[openstreetmap/openstreetmap-website] Implement JSON support for the changeset download API (PR #5973)

mmd notifications at github.com
Mon May 5 20:53:12 UTC 2025


mmd-osm left a comment (openstreetmap/openstreetmap-website#5973)

Indeed, this looks like a good option. It's probably a matter of taste to have "type": "create" or "action": "create", either way should be ok. I can see the benefit of keeping the element objects clean.

NB: I plan to use an event-based parser to avoid keeping the entire parsed document in memory. The proposed structure has some implications on how the parser needs to work for if-unused as well as the additional properties/options.

On the other hand, it's a good reminder that we can't make assumptions about the order of properties (i.e. we can't assume insertion order). I think this should still be doable.



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

Message ID: <openstreetmap/openstreetmap-website/pull/5973/c2852308362 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250505/395c70a4/attachment-0001.htm>


More information about the rails-dev mailing list