search

IgnitionModuleDevelopmentCommunity / IgnitionNode-RED

Adds support for Node-RED
Apache License 2.0
31 stars 7 forks source link

nodes are destructive #24

Closed shaqaruden closed 7 months ago

shaqaruden commented 2 years ago

All the nodes overwrite msg.payload instead of appending to it, so if you have an ignition node in your flow you have to move your existing payload out of msg.payload and then set it back after the ignition node.

iatraviscox commented 2 years ago

Yes, that is true. In the meantime, you can adjust which part of the message the result gets written to:

image

However, that is not the payload object. I suppose you would like to add the result to the existing payload object?

shaqaruden commented 1 year ago

I did start doing that but I think this request was just bring the nodes in line with how most others work and how it should work IMO.

iatraviscox commented 1 year ago

I fixed this problem in 1.5.15. They now merge results. You need to update the Ignition nodes in NodeRED.

https://www.npmjs.com/package/node-red-contrib-ignition-nodes/v/1.5.15

iatraviscox commented 7 months ago

Closing due to inactivity.