-
Notifications
You must be signed in to change notification settings - Fork 70
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
Fixes page.getPage to return a page object #137
base: master
Are you sure you want to change the base?
Conversation
Prior to this fix, the page object that would be returned would be missing functions and have properties that are unique to the PhantomJS page object. This fixes issue #131.
Seems like the Travis CI build fails to retrieve SlimerJS. |
Any progress on this? |
I think more maintainers needed. I'm very busy with main project and currently use May be @awlayton / @johntitus / ... will agree? PS. I can be back if 2.5 released with upgraded engine ariya/phantomjs#14458 |
Sorry, @awlayton is basically maintaining node-horseman himself these days On Wed, Aug 10, 2016 at 1:03 PM, Vitaly Puzrin [email protected]
|
I currently do not have much more time for this sort of thing than what I am putting towards node-horseman. I would be open to possibly maintaining this as well at some point in the future, but I am not sure when that would be. |
Hm... https://github.com/baudehlo/node-phantom-simple/graphs/contributors i don't see potential "victims" in history stat :) @Reewr may be you wish to become maintainer? |
Thanks for the offer, but due to an ongoing masters degree I'm afraid I won't have time to maintain other projects than my own. |
Updated to not cause conflicts with master (after new commits were added) |
Prior to this fix, the page object that would be returned would be missing functions and have properties that are unique to the PhantomJS page object. This fixes issue #131.
The function will, as it is suppose to, return a page object if it finds an object with the given name or it'll return null.
It needed its own special function as it needs to setup a new page, both in bridge.js and node-phantom-simple.js. Also added test to make sure it works correctly.