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

Removed _emptyIteratorStopper from response #89

Merged
merged 2 commits into from
Mar 4, 2021

Conversation

ckhmer1
Copy link
Contributor

@ckhmer1 ckhmer1 commented Mar 3, 2021

Hi,
I've removed the _emptyIteratorStopper from the response and fixed a small bug where the on status was not filled.

Claudio

@ckhmer1
Copy link
Contributor Author

ckhmer1 commented Mar 4, 2021

Hi,
I've just fixed a small bug in case no lights are configured.
Thanks

@mabunixda
Copy link
Owner

Awesome! Thanks!

@mabunixda mabunixda merged commit 951e760 into mabunixda:master Mar 4, 2021
@ckhmer1 ckhmer1 deleted the bug-fix branch March 4, 2021 07:03
mabunixda added a commit that referenced this pull request Mar 4, 2021
* master:
  fix: upgrade mustache from 4.0.1 to 4.1.0 (#85)
  Removed _emptyIteratorStopper from response (#89)
  Changed the request info message to debug (#86)
mabunixda pushed a commit that referenced this pull request Mar 4, 2021
* Removed _emptyIteratorStopper from response

* Fix big if no lights are configured

Co-authored-by: Claudio Chimera <Claudio.Chimera@gmail.com>
Signed-off-by: Martin Buchleitner <mbuchleitner@infralovers.com>
mabunixda added a commit that referenced this pull request Mar 4, 2021
* npm-update:
  fix: revert linting script
  fix: ci gh action
  fix: allow linting bypass argument 1
  feat: update to semantic versioning
  fix: upgrade mustache from 4.0.1 to 4.1.0 (#85)
  Removed _emptyIteratorStopper from response (#89)
  Changed the request info message to debug (#86)
  Bump ini from 1.3.5 to 1.3.7
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