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

People names are not diacritic neutral when adding them from Document Detail page #1479

Closed
6 tasks done
richmanrachel opened this issue Oct 19, 2023 · 7 comments
Closed
6 tasks done
Assignees
Labels
🐛 bug Something isn't working performant Tasks for or taken on by Performant
Milestone

Comments

@richmanrachel
Copy link

richmanrachel commented Oct 19, 2023

testing notes (QA) -- round two

In the QA site admin:

  • Go to the People section. In the list view, use the search bar to search for people's names who have diacritics in them. Try searching both with and without diacritics. Both should bring up the expected person.
  • The same should be true for the Places section.
  • Now try to edit a Place. Go to the section where you attach related people and places. In these autocomplete dropdowns, the searches should work the same way—both diacritic and non-diacritic searches should bring up the right results.

testing notes (QA)

In the QA site admin:

  • Try adding Related People to a Document. You should be able to successfully search their names without diacritics and case-insensitive
    • try the same when adding Related People to People and Places entries
    • note that this does not include ' and ʿ being treated the same—just diacritics
  • Try adding Related Places to Documents, People, and Places. Again, you should be able to search without diacritics and pull up the correct places

Describe the bug
While we've made the search results of the website diacritc-mark neutral (i.e. you can search for "Masliah" to get "Maṣliaḥ") the same is not currently true when adding People in the Document Detail page. This will create problems and potentially duplicates of People by making it harder for RAs to attach the correct people to documents.

Screenshots/To produce bug
If I type in "Ma" I get the following results, but if I type in "Mas" without the underdot, the results don't populate:
image

image

Expected behavior
Underdots, macrons, and the hamza/ayin notations should be ignored when typing a name to pull up an existing person (i.e. use the same logic developed for the frontend).

@blms blms added 🆕 enhancement New feature or request performant Tasks for or taken on by Performant labels Jan 4, 2024
@richmanrachel
Copy link
Author

#1499 is the same; I'll copy Ksenia's comment here and close:

Is your feature request related to a problem? Please describe.
Finding people with diacritics as part of the first letter of any of their names is hard. I cannot type "H" and get Ḥayyim b. 'Ammār al-Madīnī. For those working on PCs, this makes quick data entry a pain because we have to copy and paste the diacritics separately (there is no way to create them via a keyboard shortcut).

Describe the solution you'd like
I want to be able to type in an H and get names that start with the diacritic and the diacritic-less version of this letter (Ḥ). This would apply to other such letters (Ṣ, Ḍ, Ṭ, Ẓ and ʾ)

@kseniaryzhova
Copy link

@blms works well, thank you! Closing!

@blms blms removed the 🗜️ awaiting testing Implemented and ready to be tested label Mar 4, 2024
@blms
Copy link
Contributor

blms commented Apr 11, 2024

Reopening as it works from autocomplete, but not on admin search pages.

@blms blms reopened this Apr 11, 2024
blms added a commit that referenced this issue Apr 11, 2024
Allow diacritic-insensitive searching in admin list views for People/Places (#1479)
@blms blms added the ⚠️ tested needs attention Has been through acceptance testing and needs additional work label Apr 11, 2024
@blms blms added 🗜️ awaiting testing Implemented and ready to be tested and removed ⚠️ tested needs attention Has been through acceptance testing and needs additional work labels Apr 19, 2024
@kseniaryzhova
Copy link

kseniaryzhova commented Apr 22, 2024

@blms weirdness noted, searching "Elhanan" (no diacritics) gives me 3 results, but searching "Elḥanan" only gives me 2 even though all 3 of those results have Elḥanan in them.

Likewise, for the Places pages, diacritic search also gives you less options in the dropdown than non-diacritic search:
image

@kseniaryzhova kseniaryzhova added ⚠️ tested needs attention Has been through acceptance testing and needs additional work and removed 🗜️ awaiting testing Implemented and ready to be tested labels Apr 22, 2024
@blms
Copy link
Contributor

blms commented Apr 22, 2024

@kseniaryzhova Actually, I think that is a data issue; there is something different about the way those two names are spelled with diacritics. Try copying these two and trying them separately:

  1. Elḥanan (first two results)
  2. Elḥanan (third result)

@blms
Copy link
Contributor

blms commented Apr 22, 2024

On the plus side, that means that the diacritic-less search will find even tiny unnoticeable variants like this :)

@kseniaryzhova
Copy link

@blms data entry problem, as you said! Closing and thank you!

@blms blms added 🐛 bug Something isn't working and removed ⚠️ tested needs attention Has been through acceptance testing and needs additional work 🆕 enhancement New feature or request labels Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working performant Tasks for or taken on by Performant
Projects
None yet
Development

No branches or pull requests

3 participants