Skip to content
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

Contexts can be parsed and included in instances. Standalone deriving… #1366

Merged
merged 5 commits into from
Jun 5, 2017

Conversation

jprider63
Copy link
Contributor

… is used when a context is provided. Type variables can be included in routes/TH.

… is used when

a context is provided. Type variables can be included in routes/TH.
@snoyberg
Copy link
Member

LGTM. Can you add a comment to the changelog explaining the changes and make a minor version bump in the cabal file?

@jprider63
Copy link
Contributor Author

Done. For some reason Travis didn't test the second commit. I fixed an issue so it should work with old versions of template-haskell as well.

@psibi
Copy link
Member

psibi commented Mar 27, 2017

@jprider63 Can you document about how to use this new feature either in the cookbook https://github.com/yesodweb/yesod-cookbook or the yesod book if itsn't too much work ?

@jprider63
Copy link
Contributor Author

Sure, I can add an example to the cookbook when I have some time. Would an example subsite with type variables and contexts make sense?

@psibi
Copy link
Member

psibi commented Mar 30, 2017 via email

@jprider63
Copy link
Contributor Author

I just added the example to the cookbook and created a pull request.

@jprider63
Copy link
Contributor Author

Is there anything else you'd like me to do before merging this?

@psibi
Copy link
Member

psibi commented Jun 1, 2017

@jprider63 There seems to be merge conflict with the PR.

@jprider63
Copy link
Contributor Author

The changelog changed since I wrote this. I've bumped the version and fixed the conflict.

@psibi
Copy link
Member

psibi commented Jun 5, 2017

@snoyberg Can this be merged ?

@snoyberg snoyberg merged commit 0b1a4b1 into yesodweb:master Jun 5, 2017
@snoyberg
Copy link
Member

snoyberg commented Jun 5, 2017

LGTM, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants