Skip to content

Conversation

@catskul
Copy link

@catskul catskul commented Sep 26, 2015

I did my best to adjust for 2.0 though not fully tested.

Also I found an error that should be causing you a problem in your branch here: https://github.com/odysseyscience/newforms-bootstrap/blob/static-display/src/index.jsx#L390

@seanadkinson
Copy link

Cool, looks good. When you merge and release, I'll plug it into my project, so let me know. Thanks!

@seanadkinson
Copy link

Oh, didn't realize this was a PR into our repo. I'll merge and see if everything works for us.

seanadkinson added a commit that referenced this pull request Sep 27, 2015
@seanadkinson seanadkinson merged commit 08ed2b6 into odysseyscience:master Sep 27, 2015
@catskul
Copy link
Author

catskul commented Sep 29, 2015

I have a PR on the primary repo here: insin#9 though it appears to have some errors that I need to fix.

@insin has been focusing his attention elsewhere, but I'm hoping once the tests pass it might get merged.

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

Successfully merging this pull request may close these issues.

2 participants