Support WLED repos other than the official AirCoookie #1576
Support WLED repos other than the official AirCoookie #1576netmindz wants to merge 3 commits intofrenck:mainfrom
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
Sorry, I have no intention of support any of the WLED forks. There are too many variants out there, and above all, it will create a nightmare supporting all the tiny differences between versions and APIs. ../Frenck |
|
Adding support for looking at a different GitHub repo isn't the same as trying to support true forks that might have different APIs for WLED. MoonModules has API compatibility with WLED-AC |
|
Reopening — the idea of configurable repo for updates/releases is worth pursuing, especially now that WLED has moved to the |
|
Superseded by #2039 |
If you have devices running builds from your own fork or WLED-MM (MoonModules) then you should be able to configure the WLED add-on in HomeAssistant to check this repo for the source of your updates, rather than hard-coded to
Aircoookie/WLEDIdeally, it should look on a per-device basis to see where its release came from, but this might require some changes to WLED to present this through the API and not just as the badge on the manual OTA page - I still need to investigate this and that might be for later PR