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.

Sign up for GitHub

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

Jump to bottom

fix: use ILIKE and replace lowercase for spot category search #133

Merged
merged 2 commits into from
Feb 20, 2025
Merged

fix: use ILIKE and replace lowercase for spot category search #133

merged 2 commits into from
Feb 20, 2025

Conversation

dgd03146
Copy link
Owner

Motivation 🤔

  • Need to improve case-insensitive search for spot categories
  • Optimize query performance by using database-level functionality

Key Changes 🔑

  • Replace spot.category.toLowerCase() with SQL ILIKE operator
  • Move case-insensitive matching to database level

To Reviews 🙏🏻

  • Verify case-insensitive search works correctly
  • Check if query performance is improved

@dgd03146 dgd03146 added the 🐞 Bug Something isn't working label Feb 20, 2025
@dgd03146 dgd03146 self-assigned this Feb 20, 2025
@vercel Vercel
Copy link

vercel bot commented Feb 20, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
jung-admin ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 20, 2025 1:16pm
jung-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 20, 2025 1:16pm

@dgd03146 dgd03146 merged commit 1e02b30 into main Feb 20, 2025
2 of 5 checks passed
@dgd03146 dgd03146 deleted the release/v1.2.2 branch February 20, 2025 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@dgd03146 dgd03146

Labels
🐞 Bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dgd03146