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 Neeva.com search engine #2133

Closed
wants to merge 1 commit into from
Closed

Conversation

ElijahLynn
Copy link
Contributor

Follow up to #2088 by @PhantomLel.

Copy link
Collaborator

@Davidy22 Davidy22 left a comment

Choose a reason for hiding this comment

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

I do want to eventually shift towards the thing I mentioned in the original pull request where we have a way to enter your own search engine in preferences because I really don't want the endgame for this to be a mile long list of search engines, but I won't say no to additions. This PR does need a little revision though, because as it is currently, the search engine won't actually show up in the list of options in preferences. Need to mirror the changes in the prefs.glade file, put your addition in the same relative spot in the drop down list so that people can actually choose it. You should be able to install and test your changes to see if they work as intended with a sudo make install

A release notes file generated with make reno SLUG=neeva_search_engine or some other descriptive header filled in would also be nice to smooth the wheels on pulling this.

@PhantomLel
Copy link
Contributor

I can work on allowing users to add whatever search engine

@Davidy22
Copy link
Collaborator

Oh certainly that would be very welcome

@Davidy22
Copy link
Collaborator

Closing because superceded by #2134

@Davidy22 Davidy22 closed this Oct 20, 2022
@ElijahLynn
Copy link
Contributor Author

W00t! Yeah, much better to go with user entered, custom search engines! Thank you!

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.

3 participants