Felix Paulusma
Felix Paulusma
I believe we've opted to just use `PostCreated '[PlainText] NoContent` And add whatever instance it complains about so that it returns a 201 Created 👍 with an empty body
Making a `MimeUnrender ctyp NoContent` should be easy by enforcing the body should be empty, right? ```hs mimeUnrender _ "" = Right NoContent mimeUnrender _ _ = Left "Unexpected content"...
Using `makeRelativeToProject` will at least make it consistent between `stack` and `cabal`. Not sure what the goal was outside of this minimal reproduction?
Documentation doesn't say the `FromJSON` and `ToJSON` instances are necessary (anymore?), so it doesn't need to be changed for that per se. But more explanation of the subtle differences in...
> `incomingJSON` is already `[Value]` No, no it's not. It's `Value`, that's why it's being parsed to "just force the array" to then `setVersion` on the individual `Value`s, which then...
> The test functions like `migrateRoundTripProp` don't need a second type application. Adjust examples, but note that adding the second type gives guarantee it's migrating from the right type. This...
Can confirm, I got this too just today trying to make a PR for the [`stack`](https://github.com/commercialhaskell/stack) code base. (which uses `nightly-2022-09-05` => 9.2.4)
I had the same happen recently with 9.6.3 and 9.6.4, yeah 🤔 When working on the `password` library to update it to accept `base64-1.0`, I bumped the Stack LTS and...
@belevy I'm wondering if you couldn't just use the `JSONB` type from `Database.Esqueleto.PostgreSQL.JSON`? 🤔 And maybe also `JSONBExpr`? That way you can also use all the JSON operators while aggregating...
I still don't see why the `JSONB` newtype isn't used? - Move the `JSONB` newtype to `Database.Esqueleto.Internal.JSON` - Re-export it in `Database.Esqueleto.PostgreSQL.JSON` - Use it in `Database.Esqueleto.PostgreSQL.JSON.Experimental` - Be able...