parsonsmatt/servant-persistent

Change String to Text?

kozak opened this issue · 4 comments

kozak commented

Thanks a lot for your tutorials, they are awesome! 🥇

Would it make sense to use Text instead of String in entity definitions, so that beginners like me, won't get tripped ?

That's a great idea! Would you like to put together a PR to accomplish that?

Are we talking about this the User entity to Text rather than Strings.

I only ask because I tried to literally replace String with Text, but I'd then have to install the Data.Text package.

It sounded like this was a simpler change, so just wanted to verify if that is what is actually wanted.

Yup! That would be a good change to make.

To install Data.Text, you'll want to add that dependency to the cabal file.

Thanks @StevenXL for fixing this!