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

Prioritize amount of matches on searches when using multiple words #890

Closed
daronco opened this issue Sep 19, 2016 · 3 comments
Closed

Prioritize amount of matches on searches when using multiple words #890

daronco opened this issue Sep 19, 2016 · 3 comments
Assignees
@daronco @RafaFP
Labels
cat:RNP-2016-Q3 prio:medium type:feature
Milestone
v2.4.0

Comments

@daronco
Copy link
Member

daronco commented Sep 19, 2016

Currently, when searching for users to invite to a conference, for example, typing the user's first and last name is sometimes worse than just typing a single name.

Multiple words are split and searched with "or", so that all results that match one word or the other are shown. This is good, but the results that match both words should receive priority and be displayed at the top.

This happens for all searches in Mconf-Web and could be solved in all of them together.
@RafaFP RafaFP changed the title Prioritize exact searches when using multiple words Prioritize amount of matches on searches when using multiple words Nov 3, 2016
@RafaFP RafaFP mentioned this issue Nov 7, 2016
Merged
@daronco daronco self-assigned this Nov 7, 2016
@RafaFP
Copy link
Contributor

RafaFP commented Nov 8, 2016

When #900 and #898 get merged, we must add this implementation to their specific "search_by_terms" contexts.

daronco added a commit that referenced this issue Dec 1, 2016
@daronco
Merge branch 'prioritize-matches-search-by-terms-#890' of https://git…
6b69287 
…hub.com/RafaFP/mconf-web into RafaFP-prioritize-matches-search-by-terms-#890
daronco added a commit that referenced this issue Dec 1, 2016
@daronco
Merge PR #908 from branch 'RafaFP-prioritize-matches-search-by-terms-#…
81ed617 
…890'
@daronco daronco removed their assignment Dec 1, 2016
@daronco
Copy link
Member Author

daronco commented Dec 1, 2016

I'm reopening this because the prioritization still needs to be implemented for other resources that have the method search_by_terms: Event, Space, BigbluebuttonRecording

@daronco daronco reopened this Dec 1, 2016
@daronco daronco added this to the v2.4.0 milestone Dec 1, 2016
@RafaFP RafaFP mentioned this issue Dec 20, 2016
Merged
daronco referenced this issue Dec 21, 2016
@daronco
Merge pull request #916 from RafaFP/apply-prioritizer-all-searches-#890
b433267 
Prioritizer for search_by_terms on spaces and recordings
@daronco
Copy link
Member Author

daronco commented Dec 21, 2016

Closed by #916.
What's left to be done is related to events and will be done at #917.

@daronco daronco closed this as completed Dec 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@daronco daronco

@RafaFP RafaFP

Labels
cat:RNP-2016-Q3 prio:medium type:feature
Projects
None yet
Milestone
v2.4.0
Development

No branches or pull requests
2 participants
@daronco @RafaFP