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.

Sign up for GitHub

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

Jump to bottom

remove the usage of apollo client #63

Merged
merged 16 commits into from
Jan 19, 2021
Merged

remove the usage of apollo client #63

merged 16 commits into from
Jan 19, 2021

Conversation

wawhal
Copy link
Contributor

@wawhal wawhal commented May 15, 2020 •
edited
Loading

Changes:

  1. Remove the usage of Apollo client, thus reducing the bundle size significantly
  2. [Breaking] Change the API for using graphqurl in a script (Refer README). The API for the CLI is unchanged.

Rishichandra Wawhal and others added 12 commits April 26, 2020 10:03
new API POC
78e7b89
Merge remote-tracking branch 'origin/master' into rewrite
527509a
remove apollo client completely
3bdee7a
update readme
64d6eae
remove redundant deps
5915ce7
remove redundant imports
0aacf64
reorganise scripts
605bc34
@shraddhaag
update Library usage code snippets in README; npm audit fix (#64)
a3d6126 
* npm audit fix

* docs: update Library usage code snippets

This commit updates the function to create GraphQL client to
createClient in all code snippets under Library docs.
@shraddhaag
fix missign CLI output (#65)
168579c 
* replace introspectQuery with getIntrospectionQuery

introspectQuery is deprecated in graphql v15.0.0 in favour of
getIntrospectionQuery.

* cli: add module query.js

This commit adds module query.js to facilitate running queries.
It makes use of npm package rewire to use unexported query func
in command.js

* cli: refactor ui.js

This commit makes use of query function in ui.js to implement DRY.

* cli: update introspectionQuery func in command.js

* cli: support mutation in query.js

* cli: fix eslint errors

* cli: remove rewire

* cli: catch error while fetching introspection schema in ui.js
@shraddhaag
Add example script, update readme, use isomorphic fetch (#66)
3aa5fe1 
* add example script

* remove dependency node-fetch

* update readme

* update dependency for example script

* readme: minor fix
Bump version
4de7269
downgrade graphql version
33a8b38
@wawhal
Copy link
Contributor Author

wawhal commented Jun 12, 2020 •
edited
Loading

Some pending tasks:

  1. Ability to close a running subscription.
  2. Ability to refresh the client with new headers. This is done, just needs to be tested and documented.
  3. Test with multiple subscriptions for edgecases.

@FDiskas FDiskas mentioned this pull request Oct 8, 2020
Closed
@shraddhaag
example: update GraphQL endpoint and queries
4d2a08b 
Signed-off-by: Shraddha Agrawal <shraddha.agrawal000@gmail.com>
@wawhal wawhal merged commit 7226131 into master Jan 19, 2021
@OliverCole OliverCole mentioned this pull request Jan 19, 2021
Closed
@mushkevych mushkevych mentioned this pull request May 13, 2021
Open
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shraddhaag shraddhaag shraddhaag approved these changes

@praveenweb praveenweb Awaiting requested review from praveenweb

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wawhal @shraddhaag