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

Improve Search Function of the AGAGD #164

Open
michaelhiiva opened this issue Mar 3, 2021 · 2 comments
Open

Improve Search Function of the AGAGD #164

michaelhiiva opened this issue Mar 3, 2021 · 2 comments
Labels
bug Bugs in our AGAGD system. feature request Any new feature for the AGAGD.
Milestone

Comments

@michaelhiiva
Copy link
Contributor

Summary of the Bug

Currently the AGAGD requires a search to be in a specific format such as: [lastname], [firstname], [lastname] or [id].

The Expected Behaviour

The search should be able to handle other criteria like: "tournaments," "games vs," "by firstname" and other advanced searches.

@michaelhiiva michaelhiiva added bug Bugs in our AGAGD system. feature request Any new feature for the AGAGD. labels Mar 3, 2021
@michaelhiiva michaelhiiva added this to the Redesign milestone Apr 12, 2021
@michaelhiiva
Copy link
Contributor Author

Related #241.

abaisero added a commit to abaisero/agagd that referenced this issue Apr 5, 2022
Prior to this commit, the search query was used monolithically as a
single string to match the members' member id or their (comma-separated)
full name;  this had many shortcomings, e.g., a search query " 12345 "
would fail to match a member with id "12345", and a search query
"surname name" would fail to match a member with full name "surname,
name".  Also see usgo#164 and usgo#248.

In this commit, the search is first stripped of outer whitespace, which
helps search for member ids.  If the search query is not an id, then the
search query is split into tokens, and a match with a member is found if
all of the respective query tokens match the member's full name.  This
allows users to run queries such as "name", "surname", "name surname" or
"surname name".
@Trevoke
Copy link

Trevoke commented Jun 27, 2022

Search by ID seems broken - https://agagd.usgo.org/search/?q=14460 gives me a 500 in production

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bugs in our AGAGD system. feature request Any new feature for the AGAGD.
Projects
None yet
Development

No branches or pull requests

2 participants