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

Find a driver instance under WebElement object #15

Merged
merged 1 commit into from
Nov 19, 2016

Conversation

kejkz
Copy link
Contributor

@kejkz kejkz commented Nov 18, 2016

This PR adds the ability to fetch the driver instance directly from the WebElement object and do not try to initialize the new driver instance if _driver attribute does not exist. This is needed if custom driver instance is used while initializing the Webium page objects.

@SUNx2YCH
Copy link
Contributor

@kejkz, thanks for contribution! Looks fine for me.
Will look into Travis issues on weekend, it seems that selenium v3 ruined the build.

@kejkz
Copy link
Contributor Author

kejkz commented Nov 18, 2016

@SUNx2YCH I've actually just pushed the proposed change, with the custom installation of the geckodriver and latest Firefox (50), I hope it will work.

Thank you for the fine library

@SUNx2YCH
Copy link
Contributor

@kejkz, sorry, but I'd prefer not to use GeckoDriver on Travis until it become more stable.
Could you please remove the 2nd commit and rebase on master? See #16.

@kejkz
Copy link
Contributor Author

kejkz commented Nov 19, 2016

@SUNx2YCH sure, np

@kejkz kejkz force-pushed the find-element-driver-instance branch from 159de1d to d52db28 Compare November 19, 2016 11:43
@kejkz
Copy link
Contributor Author

kejkz commented Nov 19, 2016

@SUNx2YCH Done! Thanks again for the effort.

@SUNx2YCH SUNx2YCH merged commit 19a8af0 into wgnet:master Nov 19, 2016
@kejkz kejkz deleted the find-element-driver-instance branch November 19, 2016 11:52
@SUNx2YCH
Copy link
Contributor

@kejkz, this change is now a part of 1.2.0 release:

Thanks again for contribution.

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