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

Give Query a to_payload() method #11

Open
jwodder opened this issue May 2, 2024 · 0 comments
Open

Give Query a to_payload() method #11

jwodder opened this issue May 2, 2024 · 0 comments
Labels
library API Relating to the public library API under consideration Dev has not yet decided whether or how to implement

Comments

@jwodder
Copy link
Owner

jwodder commented May 2, 2024

  • Give Query a to_payload(&self) -> Payload method with a default implementation that wraps the text of write_graphql() in query($VARIABLES) { ... } to get the full query document

    • The write_graphql() and variables() methods will still be needed for creating batch queries.
  • Rename write_graphql() to something like write_query_content() or write_inner_query().

@jwodder jwodder added under consideration Dev has not yet decided whether or how to implement library API Relating to the public library API labels May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
library API Relating to the public library API under consideration Dev has not yet decided whether or how to implement
Projects
None yet
Development

No branches or pull requests

1 participant