-
Notifications
You must be signed in to change notification settings - Fork 368
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
Better monads #1464
Better monads #1464
Conversation
* Also removes deprecation from `timeField` * Also mildly discourages using `timeFieldTypeText`
Should the docs (“While this is simply a @WriterT@”) be updated too? |
Hah, I didn't expect anyone to be reviewing this. Good catch. I'm not quite sure which (if any) of these experiments will end up making it onto master, so feedback on whether this is worth it at all is certainly welcome :). I'm currently experimenting with a pretty crazy subsite rewrite based on all of this, but it's not ready to be pushed, and I probably won't finish it until at least next week due to travel. |
…to better-monads
While you are doing some breaking changes, I think this is a good time to fix this also: #1418 (comment) |
In the implementation of contentTypeTypes make use of simpleContentType.
…into better-monads
This is much more consistent than suddenly using a ReaderT for subsites. Thanks to @jprider63 for the inspiration for this, I think it cleans things up a lot!
Update `mkYesodWith` and refactor so that `mkYesod` uses the context parser
Technically unnecessary since it's inherited from conduit, but this is more explicit.
No description provided.