@hlfan commented on this pull request.


> @@ -80,6 +80,11 @@ OSM = {
 
     const hash = OSM.parseHash(location.hash);
 
+    const geoURI = Object.fromEntries(new URLSearchParams(
+      params.geouri?.replaceAll(/[?;]/g, "&")?.replace(":", "=")

> Are you going to look at anything past the first semicolon?

In RFC 5870 uncp comes after crsp, so to parse 
`geo:57.6247,-3.6845;crs=wgs84;u=100`, I need to.
> like `:`, `&` is possible inside geouri parameters

Well then `new URL()` isn't going to help me either.

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

Message ID: 
<openstreetmap/openstreetmap-website/pull/5736/review/2658698...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to