-
Notifications
You must be signed in to change notification settings - Fork 194
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
Upgrade Capybara #305
Comments
Yeah, a lot of broken things. Sorry about that. Trying to get back on track. Paying $100 to anyone who puts Monologue back on track with Rails 5 support. Closing but see #314 |
I am easily bought with $100. |
@illegalnumbers hehe :) Does that mean you want to give it a shot and "make Monologue great again" and make it work with Rails 5 and all that jazz? Would be awesome! |
I will see if I can find some time. Unfortunately it seems my promises these days to do side coding tend to be for naught so I avoid all promises now. |
Sure, no worry :) Just post here if you do start the work, to avoid someone else to start it at the same time. Thanks! |
Well, not here but in #314 in fact. |
Capybara is at a very old version and should be upgraded. This will break many tests. I tried a brief upgrade but some of the things that the tests were doing I couldn't fix, namely window switching to the new window switching API and you'll need to add in some id's on content to find them.
The text was updated successfully, but these errors were encountered: