Skip to content
This repository was archived by the owner on Apr 9, 2024. It is now read-only.

update for latest selenium#106

Closed
PaleNeutron wants to merge 1 commit into
altair-viz:masterfrom
PaleNeutron:master
Closed

update for latest selenium#106
PaleNeutron wants to merge 1 commit into
altair-viz:masterfrom
PaleNeutron:master

Conversation

@PaleNeutron
Copy link
Copy Markdown

find_element_by_id type methods are removed from selenium science version 4.30

@andyreagan
Copy link
Copy Markdown

@jakevdp what is needed to merge in these updates? if there are tests or other things that are needed I'd be happy to help.

@rencryptofish
Copy link
Copy Markdown

hi @jakevdp, do you happen to know if we can merge this in, tested locally seems fine and minor fix

@normanrz
Copy link
Copy Markdown

It would be fantastic to get this merged 🚀

@PaleNeutron PaleNeutron closed this Dec 8, 2023
@fleimgruber
Copy link
Copy Markdown

@PaleNeutron Just curious, are there plans to support selenium >=4.30 or is this done in another PR?

@PaleNeutron
Copy link
Copy Markdown
Author

PaleNeutron commented Dec 8, 2023

@fleimgruber, No, but I think this library will be deprecated in the feature since https://github.com/vega/vl-convert is a much better alternative.

@fleimgruber
Copy link
Copy Markdown

I see, thanks for the heads up and the link!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants