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

Setup react adapters #171

Merged
merged 3 commits into from Feb 6, 2019

Conversation

Projects
None yet
1 participant
@andyrichardson
Copy link
Contributor

andyrichardson commented Feb 5, 2019

Todo:

  • Remove "monolith" react client (no longer using term client either - woop woop!)
  • Use alternate implementation of hooks for new client lib
  • Add render props implementations for Mutation, Query, and Subscription
  • Remove redundant connecting functions and instead expose Provider/Context
  • Add functional tests for client utils
  • Add hooks and render prop implementations to example
  • Add e2e tests for both react implementations

andyrichardson and others added some commits Feb 3, 2019

setup react adapters init
Co-authored-by: blainekasten <blainekasten@gmail.com>

@andyrichardson andyrichardson force-pushed the setup-react-adapters branch from 1905a75 to ca0f008 Feb 5, 2019

@andyrichardson andyrichardson merged commit 4d0a6b9 into v1 Feb 6, 2019

1 of 4 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/travis-ci/push The Travis CI build failed
Details
coverage/coveralls Coverage decreased (-5.0%) to 36.73%
Details
security/snyk - package.json (kitten) No new, high severity issues
Details

@kitten kitten deleted the setup-react-adapters branch Feb 14, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment