Skip to content

fix: case sensitive search in postgres#1801

Merged
rolznz merged 1 commit intogetAlby:masterfrom
krrish-sehgal:fix/case-insensitive-search
Oct 14, 2025
Merged

fix: case sensitive search in postgres#1801
rolznz merged 1 commit intogetAlby:masterfrom
krrish-sehgal:fix/case-insensitive-search

Conversation

@krrish-sehgal
Copy link
Copy Markdown
Contributor

fixes #1779

Screen.Recording.2025-10-01.at.5.27.36.PM.mov

@reneaaron reneaaron requested a review from Copilot October 8, 2025 13:52
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes case-sensitive search functionality in PostgreSQL by implementing database-specific search operators. The change ensures that app name searches are case-insensitive across both SQLite and PostgreSQL databases.

  • Adds conditional logic to use PostgreSQL's ILIKE operator for case-insensitive searches
  • Maintains existing LIKE behavior for SQLite and other databases
  • Preserves the existing prefix matching pattern with % wildcard

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@krrish-sehgal
Copy link
Copy Markdown
Contributor Author

krrish-sehgal commented Oct 13, 2025

@reneaaron and @rolznz , any updates for this?

Copy link
Copy Markdown
Contributor

@rolznz rolznz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK, thanks!

@rolznz rolznz merged commit 0d28c0d into getAlby:master Oct 14, 2025
8 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Command search for apps should be case insensitive

3 participants