Skip to content
This repository has been archived by the owner on Apr 28, 2022. It is now read-only.

Commit

Permalink
v2.1.9
Browse files Browse the repository at this point in the history
- Bump ArduinoJson library requirment for v6.7.0-beta (better memory management)
- Update version
- Update version info
  • Loading branch information
debsahu committed Dec 13, 2018
1 parent 5758aa5 commit 228666a
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 3 deletions.
4 changes: 2 additions & 2 deletions Arduino/McLighting/McLighting.ino
Original file line number Diff line number Diff line change
Expand Up @@ -53,8 +53,8 @@
#endif

#ifdef ARDUINOJSON_VERSION
#if !(ARDUINOJSON_VERSION_MAJOR == 6 and ARDUINOJSON_VERSION_MINOR == 6)
#error "Install ArduinoJson v6.6.0-beta"
#if !(ARDUINOJSON_VERSION_MAJOR == 6 and ARDUINOJSON_VERSION_MINOR == 7)
#error "Install ArduinoJson v6.7.0-beta"
#endif
#endif

Expand Down
2 changes: 1 addition & 1 deletion Arduino/McLighting/version.h
Original file line number Diff line number Diff line change
@@ -1 +1 @@
#define SKETCH_VERSION "2.1.8"
#define SKETCH_VERSION "2.1.9"
6 changes: 6 additions & 0 deletions Arduino/McLighting/version_info.ino
Original file line number Diff line number Diff line change
Expand Up @@ -35,4 +35,10 @@
* 11 Dec 2018 v 2.1.8
* - Fix Auto-Discovery for HA version >= 0.84 #286
* - Fix #283
*
* 13 Dec 2018 v 2.1.9
* - HA is not getting the correct animation name being run, boils down to changes to ArduinoJson library
* - Bump ArduinoJson library requirment for v6.7.0-beta (better memory management)
* - sendState() needs extra memory for jsonBuffer
* - sensState() effect can be sent directly instead of copying from PROGMEM
*/

2 comments on commit 228666a

@toblum
Copy link
Owner

@toblum toblum commented on 228666a Dec 16, 2018

Choose a reason for hiding this comment

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

HI @debsahu,

I tested the 2.1.9 and it works good for me. I just had to change the platformio.ini for ArduinoJSON 6.7.0.
I noticed that there appers a mode 46 named "null". I can't see where it comes from. Must come from WS2812FX. Do you experience the same?

Regards
Tobias

@debsahu
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok all this is coming from memory management changes to ArduinoJSON v6.7.0. Will push a change!

Please sign in to comment.