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

IBX-9370: Aligned UDW-searchterm with main search #1422

Open
wants to merge 1 commit into
base: 4.6
Choose a base branch
from
Open

IBX-9370: Aligned UDW-searchterm with main search #1422

wants to merge 1 commit into from

Conversation

reithor
Copy link
Contributor

@reithor reithor commented Jan 16, 2025

🎫 Issue IBX-9370

Currently main search and UDW search behave differently:

  • UDW search adds a trailing wildcard * to the search term
  • Main search does not add a trailing wildcard

This has some pitfalls:

  • UDW search and main search will potentially provide different results.
  • Wildcard character might be different across search engines or wildcards might not be supported at all.

PR is removing the wildcard.

@reithor reithor force-pushed the ibx-9370-align-udw-searchterm-with-main-search branch from 43907b4 to c7b82cf Compare January 16, 2025 08:56
@reithor reithor changed the base branch from main to 4.6 January 16, 2025 08:57
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@reithor reithor marked this pull request as ready for review January 16, 2025 09:45
@reithor reithor requested a review from a team January 16, 2025 09:45
@reithor reithor requested a review from dew326 January 16, 2025 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dew326 dew326 Awaiting requested review from dew326

Assignees

@dew326 dew326

Labels
Ready for review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@reithor @dew326