Skip to content

Change request library #2

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

Merged
merged 4 commits into from
Nov 13, 2024
Merged

Change request library #2

merged 4 commits into from
Nov 13, 2024

Conversation

mdervisaygan
Copy link
Member

  • Removed Axios usage and added native node:fetch usage support
  • Some design changes have been made to the readme file.

…ent to scrape.do. It can be used in case of need for values such as server-timing, scrape.do-rid in the header that are not provided with the library.

- Added 2 new tests to check if the sdo values in the header are returned correctly and to check if the html returns an error.
- playWithBrowser Test has been made more detailed.
@mdervisaygan mdervisaygan force-pushed the change-request-library branch from 7312fdd to 1da76a6 Compare November 13, 2024 01:34
@mdervisaygan
Copy link
Member Author

@mertbekci

  • Added the ability to return the header information in the request sent to scrape.do. It can be used in case of need for values such as server-timing, scrape.do-rid in the header that are not provided with the library.
  • Added 2 new tests to check if the sdo values in the header are returned correctly and to check if the html returns an error.
  • playWithBrowser Test has been made more detailed.
  • Made some improvements to the readme file and added additional information

Copy link
Member

@mertbekci mertbekci left a comment

Choose a reason for hiding this comment

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

🚀

@mertbekci mertbekci merged commit d76900c into main Nov 13, 2024
1 check passed
@mertbekci mertbekci deleted the change-request-library branch November 13, 2024 09:41
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.

2 participants