Lenient combinator infers to "Either String"

[domenkozar]

Using ReqBody' '[Required, Lenient] '[JSON] Events

Expected according to docs is Either Text.

[phadej]

It's indeed an inconsistency. one need to modify HasServer (ReqBody ' ...) instance

[mcgizzle]

Hey I'm a heavy user of Servant and would like to get involved in its development.

I think this might be a good first issue? But might need some extra info on what needs to be done before I can start hacking.

[alpmestan]

Hello @McGizzle! This indeed should be a great first issue, the patch for this is quite trivial.

All we need is to change Either String to Either Text in https://github.com/haskell-servant/servant/blob/master/servant-server/src/Servant/Server/Internal.hs#L581, and adapt whatever breaks because of this :-)

[mcgizzle]

Great! I should have some spare time at the weekend so will do that and make a PR :)