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.

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

Add better support for applications search #1092

Merged
merged 3 commits into from
Apr 12, 2021
Merged

Conversation

bpsushi
Copy link
Collaborator

@bpsushi bpsushi commented Mar 30, 2021

Plain user input is now converted to tsquery with prefix support
Add tests for improved search

resolves #969

Plain user input is now converted to tsquery with prefix support

Add tests for improved search
@bpsushi bpsushi requested a review from pbn4 March 30, 2021 12:30
@exygy-dev exygy-dev temporarily deployed to bloom-refere-fix-master-bbf4xy March 30, 2021 12:30 Inactive
@bpsushi bpsushi linked an issue Mar 30, 2021 that may be closed by this pull request
Copy link
Contributor

@pbn4 pbn4 left a comment

Choose a reason for hiding this comment

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

Hey @bpsushi this change stops searching e.g. the email address of applicant. Also I think the problem was that john wouldn't get a match when data contains John and actually Joh would be a successful partial match so I think the test also requires an adjustment to consider lower case.

Bart added 2 commits April 9, 2021 12:09
Add search by all applicant fields

Add search by alternateContact emai

Add escape of special characters to plainto_tsquery, by default they was changed to AND operators
@exygy-dev exygy-dev temporarily deployed to bloom-refere-fix-master-kkz7vm April 9, 2021 10:11 Inactive
@netlify
Copy link

netlify bot commented Apr 9, 2021

Deploy preview for clever-edison-cd22c1 ready!

Built with commit 835929d

https://deploy-preview-1092--clever-edison-cd22c1.netlify.app

@pbn4 pbn4 merged commit 23ae320 into master Apr 12, 2021
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.

Application filter doesn't work properly
3 participants