@1ec5 commented on this pull request.
> @@ -68,7 +68,7 @@ export default function (map) {
$("#maxlat").val(ne.lat);
$("#export_overpass").attr("href",
-
`https://overpass-api.de/api/map?bbox=${sw.lng},${sw.lat},${ne.lng},${ne.lat}`);
+ `${OSM.OVERPASS_URL.replace(/\/interpreter$/,
"/map")}?bbox=${sw.lng},${sw.lat},${ne.lng},${ne.lat}`);
Would it be better to generalize the `overpass_url` setting to specify just
the base URL without the api/interpreter/ path? I don’t think Overpass lets
instances customize these paths anyways.
https://github.com/openstreetmap/openstreetmap-website/blob/173885c17d91c4a2ceb70f7a4e911f2b250628ef/config/settings.yml#L132-L133
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/7208#pullrequestreview-4630072620
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/7208/review/[email protected]>_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
