jarvis icon indicating copy to clipboard operation
jarvis copied to clipboard

Offer to search Google when no search results are returned

Open jonstodle opened this issue 7 years ago • 5 comments

Adds a function on IQueryProvider to supply a "fallback result" which supplies a generic search with the provider. This makes it possible to add a setting specifying a desired fallback provider in the user settings later.

This closes #8.

jonstodle avatar Jan 27 '18 22:01 jonstodle

I see your point about not every provider being a global provider. Do you think the IsGlobalProvider should be on IQueryProvider or some other interface?

jonstodle avatar Jan 29 '18 22:01 jonstodle

Maybe not a property, but extended IQueryProvider -> IGlobalQueryProvider. Tho it depends. Need more people opinion on this.

reinismu avatar Jan 29 '18 22:01 reinismu

Yeah, that's more like what I was leaning towards too.

Let's get some more opinions, though 👍

jonstodle avatar Jan 29 '18 22:01 jonstodle

Nice initiative, but I think this PR strays from the original issue which was that the Google provider should allow you to search for things not suggested by the Google web service.

Personally I think that each provider should be responsible for whether or not to show fallback values, and not done globally.

patriksvensson avatar Jan 30 '18 07:01 patriksvensson

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Aug 24 '22 07:08 CLAassistant