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/c2852308...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
