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

Fixes page.getPage to return a page object #137

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

Fixes page.getPage to return a page object #137

wants to merge 3 commits into from

Conversation

Reewr
Copy link

@Reewr Reewr commented Jun 9, 2016

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.

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.
@Reewr
Copy link
Author

Reewr commented Jun 9, 2016

Seems like the Travis CI build fails to retrieve SlimerJS.

@Reewr
Copy link
Author

Reewr commented Aug 8, 2016

Any progress on this?

@puzrin
Copy link
Contributor

puzrin commented Aug 10, 2016

I think more maintainers needed. I'm very busy with main project and currently use navit with electron adapter.

May be @awlayton / @johntitus / ... will agree?

PS. I can be back if 2.5 released with upgraded engine ariya/phantomjs#14458

@johntitus
Copy link

Sorry, @awlayton is basically maintaining node-horseman himself these days
as I don't have time.

On Wed, Aug 10, 2016 at 1:03 PM, Vitaly Puzrin notifications@github.com
wrote:

I think more maintainers needed. I'm very busy with main project and
currently use navit with electron adapter.

May be @awlayton https://github.com/awlayton / @johntitus
https://github.com/johntitus / ... will agree?

PS. I can be back if 2.5 released with upgraded engine
ariya/phantomjs#14458 ariya/phantomjs#14458


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#137 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAkB_jma0SOWuXPyxVMP8h1NajrAAM0Yks5qegRkgaJpZM4IyNSN
.

@awlayton
Copy link

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.

@puzrin
Copy link
Contributor

puzrin commented Aug 11, 2016

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?

@Reewr
Copy link
Author

Reewr commented Aug 11, 2016

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.

@Reewr
Copy link
Author

Reewr commented Oct 7, 2016

Updated to not cause conflicts with master (after new commits were added)

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.

None yet

4 participants