rtfeldman / elm-iso8601-date-strings

Convert between ISO-8601 date strings and Time.Posix
https://package.elm-lang.org/packages/rtfeldman/elm-iso8601-date-strings/latest
BSD 3-Clause "New" or "Revised" License
34 stars 21 forks source link

Unhelpful TODO output from decoder failure #25

Closed Janiczek closed 3 years ago

Janiczek commented 5 years ago

We've hit this in our app:

https://ellie-app.com/66WPxcDYHDHa1

Json.Decode.decodeString Iso8601.decoder "\"2010-09-31T14:29:25.01235Z\""
--> Err (Failure "TODO deadEndsToString" <internals>)

That message originates in elm/parser, and the best way forwards would probably be for elm/parser to either implement it or stop exposing it, but right now it's probably on the users of elm/parser to not use it and instead provide their own implementation.

rtfeldman commented 3 years ago

Fixed in 1.1.4 thanks to @jamesrweb's PR!