Skip to content

Conversation

@bennett-sh
Copy link
Contributor

No description provided.

cibere
cibere previously requested changes Feb 19, 2025
@cibere
Copy link
Contributor

cibere commented Feb 19, 2025

For your repository name, the standard is to name it Flow.Launcher.Plugin.PluginName (Flow.Launcher.Plugin.YOURLS in this case). Thus, it is up to you if you want to just fix the wrong urls or rename your repository and update the other urls

auto-merge was automatically disabled February 19, 2025 19:22

Head branch was pushed to by a user without write access

@bennett-sh bennett-sh requested a review from cibere February 19, 2025 19:24
@cibere cibere dismissed their stale review February 19, 2025 19:53

new changes

cibere
cibere previously approved these changes Feb 19, 2025
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 it still needs to be tested

@jjw24
Copy link
Member

jjw24 commented Mar 4, 2025

Hi @cibere, what remaining do we need to test with this plugin?

@cibere
Copy link
Contributor

cibere commented Mar 4, 2025

Hi @cibere, what remaining do we need to test with this plugin?

Just manual testing with flow itself. I've already checked everything else, and it looks good to me

@Jack251970 Jack251970 changed the title add YOURLS integration plugin New plugin: YOURLS integration plugin Mar 7, 2025
@jjw24 jjw24 enabled auto-merge March 29, 2025 23:36
@jjw24 jjw24 merged commit 740e5fa into Flow-Launcher:plugin_api_v2 Mar 29, 2025
6 checks passed
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