Skip to content

Clean up GraphQL module#439

Merged
madsmtm merged 4 commits intomasterfrom
graphql-cleanup
Jul 2, 2019
Merged

Clean up GraphQL module#439
madsmtm merged 4 commits intomasterfrom
graphql-cleanup

Conversation

@madsmtm
Copy link
Copy Markdown
Member

@madsmtm madsmtm commented Jul 2, 2019

  • Removed the GraphQL class in favor of helper functions
  • Use natural _graphql namespace

This changes the signature of Client.graphql_requests to take dictionaries, but I don't think it matters, since people could use it anyhow (the GraphQL object was private API)

@madsmtm madsmtm self-assigned this Jul 2, 2019
@madsmtm madsmtm merged commit e670c80 into master Jul 2, 2019
@madsmtm madsmtm mentioned this pull request Jul 2, 2019
@madsmtm madsmtm deleted the graphql-cleanup branch July 3, 2019 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant