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

Fix deprecation warning switch_to_frame #599

Closed
wants to merge 3 commits into from
Closed

Fix deprecation warning switch_to_frame #599

wants to merge 3 commits into from

Conversation

ghost
Copy link

@ghost ghost commented May 4, 2016

Fix deprecation warning switch_to_frame
since selenium release 2.41.0 this should be switch_to.frame

Evert Kasteel added 3 commits May 4, 2016 09:54
Raise selenium version to 2.41.0
change deprecated switch_to_
change deprecated switch_to_
@aaltat
Copy link
Contributor

aaltat commented Aug 29, 2016

Sorry for a long delay in the answer but are you interested to get this to the master? If you are this requires some thoughts.

  1. Would you happen to remember, in which version the API was changed?
  2. We support selenium starting from 2.32.0 and if the API was changed after version, we should not directly use the new API, instead we should do try/expect block and expect proper error.

@aaltat
Copy link
Contributor

aaltat commented Dec 29, 2016

We are planning some architecture changes. This architecture change will most likely change the current code base in such way that the PR will cause conflicts and therefore cannot be merged directly to the master.

We don't have a excat date for the architecture change, but most likely it will happen in early next year. If you want this PR to be merged in the master before the architecture change, now is the time to improve the PR and get it merged to the master.

@ghost
Copy link
Author

ghost commented Jan 9, 2017

no, the changes are minor and can be redone after the architecture changes

@ghost
Copy link
Author

ghost commented Jan 9, 2017

Close pull request (not merged)

@ghost ghost closed this Jan 9, 2017
This pull request was closed.
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.

1 participant