[openstreetmap/openstreetmap-website] Remove Geonames and geocoder.ca (PR #3886)
Andy Allan
notifications at github.com
Wed Jan 11 10:58:48 UTC 2023
> Ultimately this is a very small step towards replacing GeocoderController with client-side parsing of Nominatim responses, which would make them easier to use in other contexts
On this topic, I'm not in favour. Moving more things to our home-grown javascript isn't an approach that I support. In particular, we end up creating own own (poor) html templating approaches, a client-side translation system, and this just duplicates what we already have serverside. My preferred approach for these things is to supply the client with "[html over the wire](https://hotwired.dev/)", use rails views to construct the html, and so on.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/3886#issuecomment-1378573339
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/3886/c1378573339 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20230111/a637e303/attachment.htm>
More information about the rails-dev
mailing list