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

[gql_http_link] Consider adding more context options #117

Open
micimize opened this issue Jun 8, 2020 · 0 comments
Open

[gql_http_link] Consider adding more context options #117

micimize opened this issue Jun 8, 2020 · 0 comments

Comments

@micimize
Copy link
Contributor

micimize commented Jun 8, 2020

  • should/can other links add headers?
  • should more http config options (like method) just be context? That way, useGETForQueries could eventually be a standalone a context update operation

For reference, I think the entire graphql/client.dart HttpConfig can be overridden through context, which is roughly:

class HttpQueryOptions {
  bool includeQuery;
  bool includeExtensions;
}


class HttpConfig {
  HttpQueryOptions http;
  Map<String, dynamic> options;
  Map<String, dynamic> credentials;
  Map<String, String> headers;
}
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

No branches or pull requests

1 participant