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

Add my changes to the code #18

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Thomas-Oliv
Copy link

Hi, this is my first time really using github so sorry If I made I mistake during the pull request.
Your library for node red is awesome but I felt like the ethernet-in node was lacking in features, specifically towards handshaking between the PLC and the reader/writer. I added another option so that several values can be sent at once at the update of only a single value. When the selected value which I refer to as the flag is changed, all the variables within the configuration node for that node are sent. Rather than those values being in msg.payload, I moved them to msg.values and I kept msg.payload reserved to store the value of the flag, although this could be reversed. Let me know if you guys need anything else from me.

Thanks,

AverageWeeaboo

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

Successfully merging this pull request may close these issues.

1 participant