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

[BUG] Keywords not used for search when adding module #2775

Closed
2 tasks done
MeestorX opened this issue Feb 28, 2024 · 4 comments
Closed
2 tasks done

[BUG] Keywords not used for search when adding module #2775

MeestorX opened this issue Feb 28, 2024 · 4 comments
Labels
BUG Something isn't working
Milestone

Comments

@MeestorX
Copy link

MeestorX commented Feb 28, 2024

Is this a bug in companion itself or a module?

  • I believe this to be a bug in companion

Is there an existing issue for this?

  • I have searched the existing issues

Describe the bug

In manifest.json there are products and keywords sections.
Keeping products short makes things cleaner in the GUI when looking at presets and things.
The list of supported devices or types of devices can be listed in the keywords section, but the entries this section don't look like they are being searched in the add connection search.

Steps To Reproduce

No response

Expected Behavior

The search should not only look at module names, it should look at keywords too. (I mean, they are called "keywords", right?) ;-)

Environment (please complete the following information)

- OS:
- Browser:
- Companion Version:

Additional context

It LOOKS like it should be searching the keywords, but it doesn't...

const searchResults = filter
? fuzzySearch(filter, allProducts, {
keys: ['product', 'name', 'manufacturer', 'keywords'],
threshold: -10_000,
}).map((x) => x.obj)
: allProducts

Is this the same properties as in manifest.json? e.g. in that file the property is products not product

@MeestorX MeestorX added the BUG Something isn't working label Feb 28, 2024
@jswalden
Copy link
Contributor

jswalden commented Mar 1, 2024

https://github.com/bitfocus/companion-module-base/wiki/manifest.json says "Keywords to allow users to more easily find your module by searching", so yeah, seems pretty clear this is expected to work. I'll verify it doesn't -- searching for "Camera" doesn't bring up the PTZOptics Visca module that has that string in keywords.

@peternewman
Copy link
Contributor

It's also broken in the same way on the connections webpage:
https://bitfocus.io/connections

@Julusian
Copy link
Member

Julusian commented Mar 6, 2024

this is fixed now
@willosof can you look into this for the website?

@Julusian Julusian closed this as completed Mar 6, 2024
@MeestorX
Copy link
Author

MeestorX commented Mar 6, 2024

If you're editing the website anyway, you might want to fix the sorting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BUG Something isn't working
Projects
Status: Done
Development

No branches or pull requests

4 participants