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

Fix IRMQTTServer example to work with GPIO0 as IR_RX #608

Merged
merged 1 commit into from
Jan 30, 2019

Conversation

Bajzathd
Copy link
Contributor

After hours of debugging, I have finally found why my receiver does not work, and I would like to save the next poor guy's time 😄

@crankyoldgit crankyoldgit self-assigned this Jan 30, 2019
@crankyoldgit
Copy link
Owner

Nice catch! Thanks very much for the PR/fix. I'm sorry it caused you grief. That would have been pain to debug. :-(

@crankyoldgit crankyoldgit merged commit 39ee30a into crankyoldgit:master Jan 30, 2019
crankyoldgit added a commit that referenced this pull request Feb 7, 2019
_v2.5.5 (20190207)_

**[Bug Fixes]**
- Fix decoding of Samsung A/C Extended messages. (#610)
- Fix IRMQTTServer example to work with GPIO0 as IR_RX (#608)
- Fix incorrect #define usage. (#597, #596)

**[Features]**
- Add deep decoding/construction of Daikin2 messages (#600)
- Added Old Vestel A/C support (56 Bits) with full functions. (#607)

**[Misc]**
- Add ControlSamsungAC example code. (#599)
- Add how to send a state/air-con to IRsendDemo.ino (#594)
@crankyoldgit crankyoldgit mentioned this pull request Feb 7, 2019
crankyoldgit added a commit that referenced this pull request Feb 7, 2019
_v2.5.5 (20190207)_

**[Bug Fixes]**
- Fix decoding of Samsung A/C Extended messages. (#610)
- Fix IRMQTTServer example to work with GPIO0 as IR_RX (#608)
- Fix incorrect #define usage. (#597, #596)

**[Features]**
- Add deep decoding/construction of Daikin2 messages (#600)
- Added Old Vestel A/C support (56 Bits) with full functions. (#607)

**[Misc]**
- Add ControlSamsungAC example code. (#599)
- Add how to send a state/air-con to IRsendDemo.ino (#594)
@crankyoldgit
Copy link
Owner

This has been included in the new v2.5.5 release of the library.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants