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

Update cookies.js for use in latest versions of pptr #78 #79

Merged
merged 4 commits into from
Oct 8, 2022

Conversation

smashah
Copy link
Contributor

@smashah smashah commented Jun 24, 2022

Relating to #78

@smashah smashah changed the title Update cdp.js for use in latest versions of pptr #78 Update cookies.js for use in latest versions of pptr #78 Jun 24, 2022
Copy link

@quibaritaenperdresatrompe quibaritaenperdresatrompe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice and easy, GG !

@guy-bartkus
Copy link

Merge?

@ThinhVu
Copy link

ThinhVu commented Aug 30, 2022

Merge?

You don't need to wait for the merge to use the patch. Add the line below to package.json:

"puppeteer-page-proxy": "git+ssh://git@github.com:smashah/puppeteer-page-proxy#patch-1"

@smashah
Copy link
Contributor Author

smashah commented Aug 31, 2022

thanks @ThinhVu I have merged the patch to the main branch on my fork's repo so you can also do this now:

"puppeteer-page-proxy": "smashah/puppeteer-page-proxy"

Just F.Y.I, feel free to make a PR on smashah/puppeteer-page-proxy I will try to review quickly and merge if relevant.

@TimRChen
Copy link

TimRChen commented Sep 7, 2022

wish this PR counld merge early.

@ThinhVu
Copy link

ThinhVu commented Sep 8, 2022

thanks @ThinhVu I have merged the patch to the main branch on my fork's repo so you can also do this now:

"puppeteer-page-proxy": "smashah/puppeteer-page-proxy"

Just F.Y.I, feel free to make a PR on smashah/puppeteer-page-proxy I will try to review quickly and merge if relevant.

Oh. I don't even know that we can use that short form. Nice.

@Cuadrix Cuadrix merged commit e42602b into Cuadrix:master Oct 8, 2022
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.

8 participants