Thinkerbell Adapter now expects rules as an object instead of a serialized string.

[Yoric]

This should make things a bit more readable and a bit more robust.

In particular, this should fix 1) of #520 (comment) .

[Yoric]

r? @mcav (only for the #519 patch)

[Yoric]

Also, f? @azasypkin for the API changes. See changes in the tests for details.

[mcav]

lgtm

[azasypkin]

f+, looks good to me, just one question: is it expected that source getter still returns string instead of plain object?

https://local.609aebe9f46a699f80d6e0284c3334ddf879e125.box.knilxof.org:3000/api/v1/channels/get' -X PUT -H ..... --data-binary '[{"id":"thinkerbell/Everyday in the morning, camera takes a picture./source"}]'

Response:
{"thinkerbell/Everyday in the morning, camera takes a picture./source":"{\"name\":\"Everyday in the morning, camera takes a picture.\",\"rules\":[{\"conditions\":[{\"feature\":\"clock/time-of-day-seconds\",\"source\":[{\"id\":\"getter:[email protected]\"}],\"when\":{\"Geq\":{\"Duration\":28800}}}],\"execute\":[{\"destination\":[{\"id\":\"channel:[email protected]\"}],\"feature\":\"camera/store-snapshot\",\"value\":null}]}]}"}

[Yoric]

Good catch. I'll make both match.

[Yoric]

Ok, fixed.

@mcav, if you have a chance to take a look at the revised version, that would be great.

[mcav]

Okay, still looks good to me

On Jun 17, 2016, at 10:26 AM, David Rajchenbach-Teller [email protected] wrote:

Ok, fixed.

@mcav, if you have a chance to take a look at the revised version, that would be great.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

[coveralls]

Coverage decreased (-0.05%) to 32.459% when pulling f04d91d on Yoric:refactor-thinkerbell into 7958cc3 on fxbox:master.