Skip to content

Improve project search with multi-word AND logic#6

Open
khashayarghafouri wants to merge 1 commit intocobret-git:mainfrom
khashayarghafouri:multi-word-search
Open

Improve project search with multi-word AND logic#6
khashayarghafouri wants to merge 1 commit intocobret-git:mainfrom
khashayarghafouri:multi-word-search

Conversation

@khashayarghafouri
Copy link

Updated the project search to support multiple words, requiring all words to match in project name or description. Added a tooltip to the search box to inform users about the new multi-word search capability.

Updated the project search to support multiple words, requiring all words to match in project name or description. Added a tooltip to the search box to inform users about the new multi-word search capability.
Copy link
Owner

@cobret-git cobret-git left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution! However, this change makes search significantly more restrictive by requiring ALL words to match (AND logic) rather than finding substring matches. For example, searching "task pilot manager" on a project named "Task Manager" would no longer match because "pilot" is missing. This breaks the intuitive behavior users expect and makes search less forgiving overall. If you'd like to add multi-word search, could you rework this as an optional feature with OR logic instead, so it enhances rather than replaces the current behavior?

@cobret-git
Copy link
Owner

Sorry — I now understand why your request was written that way. My mistake for describing the issue incorrectly, using the wrong scenarios, and suggesting an incorrect implementation. I’ve updated the issue to reflect the correct behavior. If you’d like to adjust your code based on the updated description, you’re very welcome to do so.

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.

2 participants