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

Added support for reloading cached accessories #5

Merged
merged 2 commits into from
Sep 13, 2016

Conversation

gsxdsm
Copy link

@gsxdsm gsxdsm commented Sep 13, 2016

Updated the addAccessory logic to support loading a cached
service/accessory

Updated the addAccessory logic to support loading a cached
service/accessory
this.fanMaster = new bigAssApi.FanMaster(1); // Expect only one fan in this setup - TODO: Allow specifying in config

this.numberOfFans = config.fan_count || 1;
this.fanMaster = new bigAssApi.FanMaster(this.numberOfFans);
Copy link
Owner

Choose a reason for hiding this comment

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

👍

@sean9keenan
Copy link
Owner

This is awesome, thanks so much for doing this!
I also see that you tried to get the motion sensor working as well. I don't even remember what exactly wasn't working on that, but #3 has some discussion and I think @pedroserano made some progress on it

Converted tabs to spaces
@gsxdsm
Copy link
Author

gsxdsm commented Sep 13, 2016

Updated PR with tabs to spaces

@sean9keenan
Copy link
Owner

Great, thanks so much!

@sean9keenan sean9keenan merged commit 678021d into sean9keenan:discovery Sep 13, 2016
@sean9keenan
Copy link
Owner

I'll push merge this into master and push onto NPM once I test it for a bit 👍

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.

2 participants