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: support puppeteer 17 #78

Merged
merged 9 commits into from
Mar 7, 2024
Merged

fix: support puppeteer 17 #78

merged 9 commits into from
Mar 7, 2024

Conversation

patrickhulce
Copy link
Collaborator

update to support .frame instead of .executionContext

@clemp6r
Copy link

clemp6r commented Jan 23, 2024

It seems Puppeteer 20 works without this fix, but in order to use Puppeteer 21 we need this again.

I know this library is not longer really maintained but it would be really helpful that someone merges this.

@patrickhulce patrickhulce merged commit 761f120 into main Mar 7, 2024
4 checks passed
@patrickhulce patrickhulce deleted the fix_pptr_17 branch March 7, 2024 17:15
@patrickhulce
Copy link
Collaborator Author

@clemp6r done

@marcusds
Copy link

marcusds commented Mar 7, 2024

@patrickhulce Are you able to also change the version number of the package and publish to NPM?

@patrickhulce
Copy link
Collaborator Author

@marcusds an automated script should have already published this, are you not seeing it?

@marcusds
Copy link

marcusds commented Mar 7, 2024

@patrickhulce I see it now, not sure why it was delayed.

Thank you so much for getting this done!

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.

3 participants