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

Pponce light fix #16

Merged
merged 18 commits into from
May 23, 2017
Merged

Pponce light fix #16

merged 18 commits into from
May 23, 2017

Conversation

pponce
Copy link
Contributor

@pponce pponce commented Apr 2, 2017

If you think this solution the right approach here is the PR for it.

@pponce pponce mentioned this pull request Apr 2, 2017
@pponce
Copy link
Contributor Author

pponce commented Apr 2, 2017

PR to fix issue #15
I also included your callbackOnceHack fix that solves issue #7. That hack has fixed my homebridge crashes related to the call back issue.

@sean9keenan
Copy link
Owner

Looks good - sorry for the delay, I'm not good at keeping on top of this.

I'm going to merge this PR in, but then I'm going to squash the commits together to make the git history a little bit cleaner 😄.

Afterwards I'm going to update the npm package

@sean9keenan sean9keenan merged commit 4214313 into sean9keenan:master May 23, 2017
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