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

BROKEN: Upgrade to Drizzle React Context API #23

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

gichiba
Copy link

@gichiba gichiba commented Jul 10, 2019

So this is where I've gotten to. I started out ambitious and have ended up systematically breaking everything.

I just want to know what is going on with the drizzle instance, because as far as I can tell it's not passing the drizzle object down as it should through the components, and this causes everything to have an empty state.

I've left a bunch of comments in the code to help go through my reasoning. Might also be helpful to read the notes on the drizzle-react document:

https://github.com/trufflesuite/drizzle-react#using-the-react-context-api-for-react-v163

and although I removed it to (unsucessfully) figure out the more fundamental issues, maybe the drizzle-react-components code might give some clues as to what the hell is going on here:
https://github.com/trufflesuite/drizzle-react-components

@gichiba gichiba requested a review from lestari-thomas July 10, 2019 14:21
@gichiba gichiba added bug Something isn't working in-progress Don't merge yet! still working on it! maintenance labels Jul 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working in-progress Don't merge yet! still working on it! maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant