-
Notifications
You must be signed in to change notification settings - Fork 47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Elm 0.19 #54
Elm 0.19 #54
Conversation
It's a small change, but I find it more readable.
This more accurately reflects our intent. With `nullable` the value must decode exactly, or be `null`. With `maybe` the value must decode exactly, you get a `Nothing` even if total junk is present.
Fixed compilation warnings.
Fix HasDecoder ElmValue
Add support for Natural.
Add support for non-empty lists (ElmType).
Fix the Date encoder.
Not very pretty. Build with "rtfeldman/elm-iso8601-date-strings": "1.1.2"
@domenkozar I fixed the tests on this PR: hercules-ci#1 |
Elm 0.19 tests
Thank you. |
Will this PR be merged and the package republished soon? How can I help? |
Maybe someone should take over this package. |
No idea why I forgot to check out GitHub issues to find already present forks, but anyways I've made another fork. I've replaced elm-export with elm-bridge, but overall code quality is pretty alpha. https://github.com/k-bx/servant-elm Example CRUD I'm using as a basis: https://github.com/k-bx/servant-elm-generically |
@domenkozar would you mind making a request to take over the package and move it to your repo? Would really help avoid situations like the one I've had |
As servant-elm moved to elm-bridge I'm no longer interested in this work (although we still use it, but will transition away). |
Including all other fixes maintained by @FPtje fork. Testing/feedback welcome.
cc @krisajenkins