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

Spelling error in JS node #2

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

Conversation

arretx
Copy link

@arretx arretx commented Jul 3, 2019

So I had created a notification in the flow under "Recv access event over MQTT". It followed directly after "Check if user is known." The notification was being sent twice with two different messages. Then I noticed that line 12 of the Javascript node was supposed to define msg.payload as "" but was spelled msg.paylod, which means the return msg; line was sending the "adding event" information in addition to the access information. I changed it to payload and tested it and it works.

So I had created a notification in the flow under "Recv access event over MQTT".  It followed directly after "Check if user is known."  The notification was being sent twice with two different messages.  Then I noticed that line 12 of the Javascript node was supposed to define msg.payload as "" but was spelled msg.paylod, which means the `return msg;` line was sending the "adding event" information in addition to the access information.  I changed it to payload and tested it and it works.
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