[openstreetmap/openstreetmap-website] Add profile location (PR #5302)

David Tsiklauri notifications at github.com
Thu Feb 6 07:30:53 UTC 2025


@nertc commented on this pull request.



> +    if params[:format] == "json"
+      render :html => @results.pluck(:name).join(", ")

PR was updated. Instead of HTML now `format=json` returns JSON array. I thought about plaintext, but I think, JSON object is much more versatile. Plus, it contains all the data that is contained in the html version (without `format=json`) 

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

Message ID: <openstreetmap/openstreetmap-website/pull/5302/review/2597866235 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250205/1cbcfa40/attachment-0001.htm>


More information about the rails-dev mailing list