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

Add SearchApi as a Web Search Tool #628

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SebastjanPrachovskij
Copy link

Added SearchApi tool.

The end user can change the search engine used on the SearchApi side via the engine parameter. Organic results share the same structure across all existing and upcoming SERP engines, so we will always have a title, snippet, and link. Google, Bing, Baidu, Google News, Bing News, Google Videos, Google Scholar, and Google Patents are already supported. Yandex and Yahoo are coming soon as well.

SearchApi is committed to supporting open-source integrations. Please feel free to mention @SebastjanPrachovskij for any future PR reviews related to SearchApi. We would be glad to help.

Add support for engines, improve status code error

Remove changes in package, add engine to env params

Improve description in env example

Remove unnecessary empty line

Improve text
@nickscamara
Copy link
Member

Hey @SebastjanPrachovskij thank you for the pr! We will check it out soon!

@SebastjanPrachovskij
Copy link
Author

Hey @nickscamara

I just wanted to follow up on the status of my pull request.

I would appreciate any feedback or suggestions for the next steps whenever you can.

@nickscamara
Copy link
Member

Hey @SebastjanPrachovskij, sorry about the delay. PR itself looks good! Could you email us a test key to test and play around with it? If you could send it to [email protected] that would be amazing. Thank you!

@nickscamara nickscamara added question Further information is requested ready to merge and removed waiting-for-review labels Sep 30, 2024
@SebastjanPrachovskij
Copy link
Author

Hey @nickscamara

Just sent you an email.

@SebastjanPrachovskij
Copy link
Author

SebastjanPrachovskij commented Oct 14, 2024

Hey @nickscamara

Just pinging to follow up on PR. 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants