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

Expose fuzzy search #1002

Merged
merged 4 commits into from
Nov 22, 2024
Merged

Expose fuzzy search #1002

merged 4 commits into from
Nov 22, 2024

Conversation

ssanjay1
Copy link
Contributor

Exposing option to turn on fuzzy searching

"$startsWith": string;
"$containsAllTermsInOrder": string;
"$containsAnyTerm": string;
"$containsAllTerms": string;
"$containsAnyTerm": string | { term: string; fuzzySearch?: boolean };
Copy link
Contributor

@nihalbhatnagar nihalbhatnagar Nov 21, 2024

Choose a reason for hiding this comment

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

Can we expose somewhere what fuzzySearch means? Not sure if dev console docs is best or if we need inline.

Copy link
Contributor

Choose a reason for hiding this comment

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

Also not sure do we need @internal?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Talked offline, I think those should go in dev console for now, and then we can think about JS doc'ing all these filter types inline later

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Also, removing export for now

nihalbhatnagar
nihalbhatnagar previously approved these changes Nov 21, 2024
@bulldozer-bot bulldozer-bot bot merged commit 0c56d21 into main Nov 22, 2024
8 checks passed
@bulldozer-bot bulldozer-bot bot deleted the ssanjay/exposeFuzzy branch November 22, 2024 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants