BadgeApp: Add "search by name" about best-practices-badge HOT 5 CLOSED

I think we should have a search dialogue that allows search in these areas:

• Project name
• Project homepage URL
• Project repository URL

We could also add "Description" later, but I worry about the overhead.

For each of these search fields, we could have "begins with/contains", with "begins with" being the default. Then index those. That way, most searches will use the index. Validate the search request by ensuring that non-blank URLs start with "https?://".

We're hoping to have lots of entries, so while efficiency isn't the be-all, there are some simple measures (like indexing) that can easily help.

from best-practices-badge.

I will attempt an initial implementation that simply leverages PostgreSQL's native text search capability, i.e., just "begins with" to start.

from best-practices-badge.

@dwvisser Could you please be sure to implement autocomplete. http://www.yoniweisbrod.com/autocomplete-magic-with-rails/

from best-practices-badge.

Search itself implemented with commit d18acbf. Should we move autocomplete to a different issue?

from best-practices-badge.

Sure.

Dan Kohn mailto:[email protected]
Senior Advisor, Core Infrastructure Initiative
tel:+1-415-233-1000

On Tue, Mar 29, 2016 at 2:38 PM, David A. Wheeler [email protected]
wrote:

Search itself implemented with commit d18acbf
d18acbf.
Should we move autocomplete to a different issue?

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#9 (comment)

from best-practices-badge.