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

Update to use new Home Assistant remote learn_command api #49

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

callumgare
Copy link

@callumgare callumgare commented Mar 3, 2023

Closes #41

You can try it out here: https://callumgare.github.io/BroadlinkIRTools/

Also adds swingModes to climate, updates out-of-date node packages, and uses github actions to deploy. I know that's quite a lot of changes for one PR but since this project hasn't been touched in many years there were a bunch of changes needed to get it a bit more to date. I've tried to separate different changes in separate commits so when reviewing it's probably easiest to review each commit individually.

@kwdavidson
Copy link

If the original author has abandoned this project, you should fork it so your changes can be used by others. It would suck for all your work to sit languishing.

@callumgare
Copy link
Author

I have forked it and I intend to keep https://callumgare.github.io/BroadlinkIRTools/ up and running until either this repo is fixed or a popular maintained fork emerges. Sadly I do not have the capacity to actively maintain my fork though. Any one is who is interested in maintaining a fork is welcome to fork my fork and maintain it going forwards though.

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.

Summary - Broken and Attempted fixes
2 participants