search

elm / url

Build and parse URLs. Useful for HTTP and "routing" in single-page apps (SPAs)
https://package.elm-lang.org/packages/elm/url/latest/
BSD 3-Clause "New" or "Revised" License
75 stars 43 forks source link

Url.percentEncode also path segments #34

Closed domenkozar closed 3 years ago

domenkozar commented 5 years ago

Fixes #25

hpate-omicron commented 5 years ago

This would be great to have, it is a confusing issue to run into

domenkozar commented 4 years ago

@evancz I'd appreciate if this was merged.

hpate-omicron commented 4 years ago

This would be nice to get in line with the RFC, this is the spec section for this piece that mentions percent encoding the path segments. https://tools.ietf.org/html/rfc3986#section-3.3

joshuakb2 commented 4 years ago

Is this fix going to be merged, and if not, why not?

joshuakb2 commented 3 years ago

Is there any reason this hasn't been merged for over a year? I keep getting emails when people update the issue because they felt misled by the documentation. @domenkozar already fixed it, it just has to be merged. Is there something lacking here?

@evancz, can you help here? I don't know who else has the power to help us move forward on this issue.