Skip to content

Conversation

@taooceros
Copy link
Member

close #274
This currently only bumps the result score based on their Priority.
After #195 , a better way that won't affect results score can be done by using OrderBy and ThenBy.

@taooceros
Copy link
Member Author

The design of assigning a weight to each plugin is a bit...primitive……. If anyone would have a better design, please feel free to comment. @Flow-Launcher/team

@taooceros taooceros self-assigned this Jan 6, 2021
@jjw24 jjw24 added enhancement New feature or request review in progress Indicates that a review is in progress for this PR labels Jan 6, 2021
Copy link
Member

@jjw24 jjw24 left a comment

Choose a reason for hiding this comment

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

got some minor changes from testing

@taooceros taooceros requested a review from jjw24 January 6, 2021 10:01
jjw24
jjw24 previously approved these changes Jan 11, 2021
@jjw24 jjw24 merged commit 96fc134 into Flow-Launcher:dev Jan 11, 2021
@taooceros
Copy link
Member Author

Should we make the scoring bump higher (50 to 150? ) since sometimes plugins' scores can be low as discussion in #274?

@jjw24 jjw24 removed the review in progress Indicates that a review is in progress for this PR label Jan 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enable plugin priority

2 participants