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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pass graphql full context to custom queries #4200

Merged
merged 3 commits into from Oct 7, 2019

Conversation

@alexandrebodin
Copy link
Collaborator

alexandrebodin commented Oct 7, 2019

Description of what you did:

Fix #4116

this PR replaces #4159

My PR is a:

  • 馃挜 Breaking change
  • 馃悰 Bug fix
  • 馃拝 Enhancement
  • 馃殌 New feature

Main update on the:

  • Admin
  • Documentation
  • Framework
  • Plugin

Manual testing done on the following databases:

  • Not applicable
  • MongoDB
  • MySQL
  • Postgres
  • SQLite
@alexandrebodin alexandrebodin added this to the 3.0.0-beta.17 milestone Oct 7, 2019
@alexandrebodin alexandrebodin requested a review from lauriejim Oct 7, 2019
@alexandrebodin alexandrebodin changed the base branch from master to releases/3.0.0-beta.17 Oct 7, 2019
@alexandrebodin alexandrebodin mentioned this pull request Oct 7, 2019
3 of 13 tasks complete
@lauriejim lauriejim changed the base branch from releases/3.0.0-beta.17 to master Oct 7, 2019
@lauriejim lauriejim changed the title pass graphql full context to custom queries Pass graphql full context to custom queries Oct 7, 2019
Copy link
Member

lauriejim left a comment

LGTM!

@alexandrebodin alexandrebodin merged commit 610d200 into master Oct 7, 2019
1 check was pending
1 check was pending
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@alexandrebodin alexandrebodin deleted the fix/graphql-queries-context branch Oct 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can鈥檛 perform that action at this time.