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.

Sign up for GitHub

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

Jump to bottom

HUE fixes and HSB support #89

Closed
wants to merge 34 commits into from
Closed

HUE fixes and HSB support #89

wants to merge 34 commits into from

Conversation

altelch
Copy link
Contributor

@altelch altelch commented Feb 20, 2017

Hi,

fixed an error in JSON argument evaluation what caused wrong setting changes. Implemented full HSB support for HUE api calls (colormode hs).

Greetings,

Heiko.

altelch and others added 30 commits January 29, 2017 11:06
@altelch
Disable power saving if LED animation is active
9b892d2
@altelch
Remove yield in loop as loop contains implicit yield. Always call del…
b763413 
…ay. If sleep == 0 then delay == yield. Else delay(x) is delay and yield.
@altelch
Init sleep during setup() from sysCfg.sleep
6caa26f
@altelch
Merge https://github.com/arendst/Sonoff-Tasmota
82ca46f 
Conflicts:
	sonoff/sonoff.ino
	sonoff/xdrv_ws2812.ino
@altelch
Merge https://github.com/arendst/Sonoff-Tasmota
138364e 
Conflicts:
	sonoff/user_config.h
@altelch
Merge https://github.com/arendst/Sonoff-Tasmota
1864edb
@altelch
Merge cleanup
9349ac1
@altelch
Merge https://github.com/arendst/Sonoff-Tasmota
b4fc94d
@altelch
Support for old HUE App. Sonoff recognized as bridge, changing bright…
509e4a4 
…ness of LED works.
@altelch
HUE Gen.2 App detects Sonoff. Saving groups/scenes still missing. Gen…
1bdf78a 
….1 App can control brightness.
@altelch
Merge cleanups
9d1e12d
@altelch
Merge https://github.com/arendst/Sonoff-Tasmota
190e6b1 
Conflicts:
	sonoff/webserver.ino
	sonoff/xdrv_wemohue.ino
@altelch
Merge https://github.com/arendst/Sonoff-Tasmota
b5ce4c5
@altelch
Merge https://github.com/arendst/Sonoff-Tasmota
4d32dfb
@altelch
Merge https://github.com/arendst/Sonoff-Tasmota
528e572
@altelch
Switched to ArduinoJSON for JSON processing HUE args. Also required f…
e866cfb 
…or future enhancemends in HUE and IR send function. Initial code for IR send support via MQTT using IRremoteESP8266.
@altelch
Fixed platformio.ini
850fe03
@altelch
ir_send_command filled with life.
f32e917
@altelch
Removed COOLIX due to older IRremoteESP8266 in Travis-CI
90622e9
@altelch
Fix in GPIO definition (still broken)
9f96c3c
@altelch
Merge https://github.com/arendst/Sonoff-Tasmota
d0791c3
@altelch
Merge https://github.com/arendst/Sonoff-Tasmota
3f3f9ab 
Conflicts:
	sonoff/sonoff_template.h
@altelch
Impemented IRsend command
47d0834
First working and tested MQTT to IRremote code
6d98b6d
Removed debug output
09601a7
@altelch
Merge https://github.com/arendst/Sonoff-Tasmota
a650c12
@altelch
Merge https://github.com/arendst/Sonoff-Tasmota
5697168 
Conflicts:
	sonoff/sonoff.ino
	sonoff/sonoff_template.h
	sonoff/webserver.ino
	sonoff/xdrv_ir_send.ino
@altelch
Merge cleanup
4abcaf4
@altelch
Merge https://github.com/arendst/Sonoff-Tasmota
9d21511
@altelch
Merge https://github.com/arendst/Sonoff-Tasmota
7867d9a
@altelch
Copy link
Contributor Author

altelch commented Feb 20, 2017

Bwah, now travis-ci uses another IRremoteESP8266 than what is included in Arduino!? Doesn't matter for the HUE part.

@altelch
Copy link
Contributor Author

altelch commented Feb 20, 2017

Ok, changed platformio.ini:
lib_deps = PubSubClient@~2.6, NeoPixelBus@~2.2.6, https://github.com/sebastienwa
rin/IRremoteESP8266.git, ArduinoJSON@~5.8.3

Now the correct versions are used and the built works again.

@arendst
Copy link
Owner

arendst commented Feb 21, 2017

First looked at your the PlatformIO library suggestion. If I try it locally it fails as it needs git installed to download and install IRremoteESP8266.git so I decided to add all libraries to the repository in the lib folder.

Now platformIO works great both locally and on Travis using the supplied libs.

I still need to look at your Hue changes but have to find some time for it...

arendst added a commit that referenced this pull request Feb 21, 2017
@arendst
v3.9.20
3f73e48 
3.9.20 20170221
* Add minimal basic authentication to Web Admin mode (#87)
* Fix Hue and add HSB support (#89)
@arendst
Copy link
Owner

arendst commented Feb 21, 2017

Implemented in version 3.9.20 just released

@arendst arendst closed this Feb 21, 2017
curzon01 pushed a commit to curzon01/Tasmota that referenced this pull request Sep 6, 2018
@arendst
v3.9.20
388eea6 
3.9.20 20170221
* Add minimal basic authentication to Web Admin mode (arendst#87)
* Fix Hue and add HSB support (arendst#89)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@altelch @arendst