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

Fix multi-field search for documents #277

Merged
merged 0 commits into from
Oct 18, 2021
Merged

Fix multi-field search for documents #277

merged 0 commits into from
Oct 18, 2021

Conversation

theguy147
Copy link
Contributor

Multi-field search is currently not working in documents. See #264.

As described in that issue there is an issue when query is not being set correctly. This fix should remedy that.

Please let me know if there is any more information you need or if I should change something about this PR because it doesn't comply with you Coding Style or anything else.

@ts-thomas
Copy link
Contributor

Thanks a lot for this PR.

@ts-thomas ts-thomas merged commit 3014e80 into nextapps-de:master Oct 18, 2021
@GanserITService
Copy link

@ts-thomas when does the change take effect? I installed the whole thing with node, but still have the old files. thanks

@theguy147
Copy link
Contributor Author

@GanserITService
you can use the git branch of flexsearch if you modify the package.json of your project and change the "flexsearch" entry like this instead of a specific version:

"flexsearch": "nextapps-de/flexsearch",

However I wouldn't recommend this for production use

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.

3 participants