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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update readme docs #36

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

dimitryzub
Copy link
Contributor

@dimitryzub dimitryzub commented Sep 21, 2022

This PR focuses on:

  • fix typos.

  • remove duplicates, for example:
    image

  • text formatting, clarification.

  • remove unnecessary examples, or possibly leave them but with details disclosure element.

  • add backlinks to some terms.

  • add pagination examples for all available APIs.

Pagination examples could be placed to oobt/ folder instead to test "out of the box tests" for all APIs. However, I'm not familiar with it and how things currently work so I need a little guidance before working on it 馃檪

Copy link
Contributor

@ilyazub ilyazub left a comment

Choose a reason for hiding this comment

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

Thanks, @dimitryzub! Let's pair on this PR and mirror changes to serpapi/serpapi-python#1.

@dimitryzub
Copy link
Contributor Author

@ilyazub would love to do that :shipit:

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.

None yet

2 participants