Fix: Finding emoji that include capital letters
What does this PR do?
This Merge Request fixes #522 (closed).
I kind of hate copy-pasting .toLowerCase()
like this but apparently I'm not allowed to refactor Misskey code for ease of merging so this will have to do.
Contribution Guidelines By submitting this merge request, you agree to follow our Contribution Guidelines
-
I agree to follow this project's Contribution Guidelines -
I have made sure to test this pull request
Merge request reports
Activity
added enhancement label
- Resolved by Marie
thank you!
and yes, factoring out all the duplication, as tempting as it is, would make merges even more of a pain than they are now (I spent 6 hours between yesterday and today preparing version 2024.9.0 (!675 - merged)…)
- Resolved by dakkar
I'm assuming you tested this.
Also I'm pretty sure we can ignore ambiguous lowercasing like
I
that could be bothi
andı
(mostly because we're already ignoring such cases)
enabled an automatic merge when all merge checks for 5cebb4da pass
mentioned in commit fefdd1c1