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

Enable local site-search and support for lowercasing searches #2161

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

Conversation

atiro
Copy link

@atiro atiro commented Aug 7, 2021

A rough start at supporting local site searches (ala #96 for other search engines) , that are making use of the same query param ('q') as Google search.

Not sure if would be better as a new output instead of overriding the Google table ?

Have also added downcasing of the search term which sometimes makes results more useful for us. This might be a niche concern.

@allinurl
Copy link
Owner

allinurl commented Aug 19, 2021

Thanks a lot for submitting this! Question, would you be able to add the ability for the user to specify the query param? It would make this a lot more flexible.

@atiro
Copy link
Author

atiro commented Sep 2, 2021

Good idea, working on it!

@atiro
Copy link
Author

atiro commented Oct 17, 2021

Updated version @allinurl but not entirely sure I'm taking the best approach to save the param variations in config options (really I need to add two more as well to handle encoded). Any better option ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants