-
Notifications
You must be signed in to change notification settings - Fork 2
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
Update Griddle to the latest version compatible with the react version used in the Geppetto codebase #21
Comments
@ddelpiano after talking yesterday with @filippomc I'm wondering if this should happen by virtue of having all those components start using the listviewer Filippo is developing which will already be on the latest Griddle (which btw supports infinite scrolling), thoughts? |
@filippomc Hey Filippo. I saw you updated griddle-react to version 1.13.1 as part of PR #48 . I'm also updating the dependency to the same version as part of this card, I already did but the casper tests are failing due to controlpanel.js still needing changes to make it work with griddle 1.13.1 https://travis-ci.org/openworm/geppetto-application/builds/556094254. Question, are you planning on making changes to controlpanel.js to make it compatible with griddle-react version 1.13.1 as part of PR #48 ? I'm working on the same, don't want to step in your toes if you are already taking care of the same issue. |
@filippomc Alright, good to know . 👍 |
We'll need to port the fixes to the new infinite scrolling plugin before we can do what @filippomc is proposing. |
@gidili knows what the fixes need to be |
The text was updated successfully, but these errors were encountered: