Case Insensitive Tags - Githubissues

UCF / UCF-Announcements-Django

0 stars 0 forks source link

Case Insensitive Tags #51

Open jmbarne3 opened 4 years ago

jmbarne3 commented 4 years ago

The typeahead for taggit tags is case sensitive. We need to make sure it pulls tags in a case insensitive manner.
The taggit tag model requires unique values for the tag text, so two values like "nursing" and "Nursing" would be considered the same. We should add logic that checks for an existing tag, regardless of case, and uses that if it exists.

cjg89 commented 4 years ago

Regarding point 1: this should (hopefully) just be a matter of adding the TAGGIT_CASE_INSENSITIVE setting + setting it to True. https://django-taggit.readthedocs.io/en/latest/getting_started.html

We had a similar issue on Events at some point: https://github.com/UCF/unify-events/pull/8

jmbarne3 commented 4 years ago

I made 2 small changes in the branch I made: I added that setting, and I updated a filter statement from using a __contains to an __icontains on the typeahead view. Those two things seem to have solved the problem.