Skip to content

Create very basic class to build graphql query string#2

Open
jlward wants to merge 7 commits intomasterfrom
issue_2
Open

Create very basic class to build graphql query string#2
jlward wants to merge 7 commits intomasterfrom
issue_2

Conversation

@jlward
Copy link
Copy Markdown
Contributor

@jlward jlward commented Sep 16, 2017

No description provided.

@jlward jlward self-assigned this Sep 16, 2017
@jlward
Copy link
Copy Markdown
Contributor Author

jlward commented Sep 16, 2017

Right now, the Query class takes the query parameters in __init__ and uses __str__ to render the query (using string manipulation). Ideally, we use __init__ to store info about the query, make a new method that takes kwargs and builds the graphql query string. This way we can do things like tell the query to be pretty printed, etc. Also, we probably ought to build a tool that takes the kwargs and deals with wrapping formatting. Doing this as a single string format feels dirty.

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