Skip to content

Added rh-wled-ctr plugin#328

Merged
HazardCreative merged 2 commits intoRotorHazard:mainfrom
jrwrodgers:main
Mar 10, 2026
Merged

Added rh-wled-ctr plugin#328
HazardCreative merged 2 commits intoRotorHazard:mainfrom
jrwrodgers:main

Conversation

@jrwrodgers
Copy link
Copy Markdown
Contributor

@jrwrodgers jrwrodgers commented Jan 27, 2026

Description

Plugin to connect to WLED devices on network and issue led instructions on event actions

Checklist

  • I've added the RHFest action to my repository.
  • The actions are passing without any disabled checks in my repository.
  • I've created a github release in my repository with semver versioning.
  • I've added all the requested links below.

Note: all checks above should be passed before a pull request will be merged.

Additional information

@HazardCreative
Copy link
Copy Markdown
Member

Your workflows appear to be failing because everything in the workflow is commented out.

Copy link
Copy Markdown
Member

@klaasnicolaas klaasnicolaas left a comment

Choose a reason for hiding this comment

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

👋🏻 Hi there!

Please complete all checks to have your plugin added to the RotorHazard plugin database.

./Klaas

@jrwrodgers
Copy link
Copy Markdown
Contributor Author

thanks @HazardCreative PICNIC error oops

@jrwrodgers
Copy link
Copy Markdown
Contributor Author

@klaasnicolaas It was falling over because it needed the list "categories" in the manifest.json. I think the https://github.com/RotorHazard/plugin-template/blob/main/custom_plugins/rh_template/manifest.json and the documentation might need updating?

@klaasnicolaas
Copy link
Copy Markdown
Member

klaasnicolaas commented Jan 29, 2026

The problem is that you're running an old version of rhfest. We're already on v3, and your plugin is still using v2 (advise to use the latest tag). The documentation and plugin template are therefore up to date 😉

./Klaas

@jrwrodgers
Copy link
Copy Markdown
Contributor Author

ahhh! sorry to be so slow.. the learning curve on this is steeeeep! - I've updated and re-released

@klaasnicolaas
Copy link
Copy Markdown
Member

Not sure but maybe the LED Effects category would be better? @HazardCreative how do you think?

@HazardCreative HazardCreative merged commit 6c06a8b into RotorHazard:main Mar 10, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants