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

Fixed the context option #55

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fixed the context option #55

wants to merge 1 commit into from

Conversation

mensinda
Copy link

Readded the context option which was removed in d728f80. With this change, GraphQLView behaves as described in the documentation.

From the README:

  • context: A value to pass as the context to the graphql() function.

This should also fix #52.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 93.617% when pulling 8b9333a on mensinda:master into 4183613 on graphql-python:master.

@averypmc
Copy link

Any word on this getting merged any time soon?

@mensinda
Copy link
Author

mensinda commented Nov 1, 2018

Any feedback? Will this get merged or did I miss something?

@tractorcow
Copy link

Thanks @mensinda I've been looking for this fix. :)

@guludo
Copy link

guludo commented Jan 22, 2019

I would really like to see this being merged. @syrusakbary is there anything holding this PR from being merged?

@babaMar
Copy link
Contributor

babaMar commented Oct 22, 2019

is this going to be merged any time soon?

@mensinda
Copy link
Author

The last commit is from Jul 20, 2018, so this project looks abandoned to me.

@babaMar
Copy link
Contributor

babaMar commented Oct 22, 2019

The last commit is from Jul 20, 2018, so this project looks abandoned to me.

yeah, same fear

@messa
Copy link

messa commented Nov 5, 2019

@syrusakbary is no more active around here. Most Graphene projects have new maintainers. Try to reach to graphql-python github org maintainers on Slack: graphenetools.slack.com

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.

passing context to GraphQLView no more supported?
7 participants