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

press_builder() seems broken. Missing 3 default values on the builder #47

Open
randall-coding opened this issue Feb 8, 2023 · 1 comment

Comments

@randall-coding
Copy link

randall-coding commented Feb 8, 2023

When I use press_builder() on my script I get this error at runtime.

Error "delay: expected number, got object"

Then I update my code to be page.press_builder().delay(30.0).press()

and then I get an error

Error "noWaitAfter: expected boolean, got object"

so then I add no_wait_after() function like so page.press_builder().no_wait_after(false).delay(30.0).press()

after that I see see this error

Error "timeout: expected number, got object"

and manually adding the timeout() to the chain makes it work.

So those three default values are not being set on my press_builder() for some reason. I'm not having this issue with any other builder so far.

DISCLAIMER: I'm not 100% sure this is true on your version. I modified my playwright slightly, but I don't think I changed anything that would effect default values for press_builder().

@randall-coding randall-coding changed the title press_builder() seems broken missing 3 default values on the builder press_builder() seems broken. Missing 3 default values on the builder Feb 8, 2023
@jquesada2016
Copy link

I noticed the same issue, but on .type_builder()

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

No branches or pull requests

2 participants