[openstreetmap/openstreetmap-website] Put map key presence in js layer definitions (PR #5918)

Marwin Hochfelsner notifications at github.com
Sat Apr 12 13:10:10 UTC 2025


@hlfan commented on this pull request.



>      YAML.load_file(Rails.root.join(layers_filename))
         .reject { |layer| layer["apiKeyId"] && !Settings[layer["apiKeyId"]] }
         .map do |layer|
           if layer["apiKeyId"]
             layer["apikey"] = Settings[layer["apiKeyId"]]
             layer.delete "apiKeyId"
           end
+          layer["hasKey"] = true if keys&.include?(layer["layerId"])

I appended my changes, but the different terminologies still bug me. That, however, can be changed in another PR.

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

Message ID: <openstreetmap/openstreetmap-website/pull/5918/review/2762160476 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250412/53b59d3c/attachment.htm>


More information about the rails-dev mailing list