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

New plugin: Hoarder #474

Open
wants to merge 1 commit into
base: plugin_api_v2
Choose a base branch
from

Conversation

rverdam
Copy link

@rverdam rverdam commented Mar 2, 2025

Adds FlowLauncher plugin for searching for bookmarks on an Hoarder instance.

For more information on Hoarder: please visit https://hoarder.app/

Copy link
Contributor

@cibere cibere left a comment

Choose a reason for hiding this comment

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

LGTM, but still needs to be tested

@rverdam
Copy link
Author

rverdam commented Mar 3, 2025

LGTM, but still needs to be tested

I tested this on my local instance, do I need to provide evidence before the PR is merged or what is the normal process?

@cibere
Copy link
Contributor

cibere commented Mar 3, 2025

LGTM, but still needs to be tested

I tested this on my local instance, do I need to provide evidence before the PR is merged or what is the normal process?

The normal process is a team member checking your repo and source code, then downloading your plugin to test it. So before this plugin can be added, me or another team member needs to test the plugin to ensure it works properly.

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.

3 participants