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 ability to force a hub to send the payload #72

Closed
wants to merge 1 commit into from

Conversation

hashworks
Copy link

When msg.payload.forceSend is set the hub will not determine if the state of the node changed, forcing a send regardless of it.

This is useful when the actual state of the device changes without knowledge of the hub.

When `msg.payload.forceSend` is set the hub will not determine if the state of the
node changed, forcing a send regardless of it.
@datech
Copy link
Owner

datech commented Oct 12, 2019

Hello, @hashworks.

Thanks for your contribution. I want to sheet some light on state change limitation. It was intentionally created to prevent infinite loops if hub output is used as input.

I was thinking to revise the list of input options to:
No
Process
Process and output
Process and output on state change

Would it be helpful for your cases?

@hashworks
Copy link
Author

Since I've no inputs that could fire twice (or create infinite loops), yes. If one needs both input options one could always create two hubs.

@datech
Copy link
Owner

datech commented Oct 12, 2019

Ok, I will add it in v0.1.10

@Barabba11
Copy link

How is useful to have 2 hubs? Does not has sense to drag 2 hubs on flow? Thanks

@hashworks
Copy link
Author

How is useful to have 2 hubs? Does not has sense to drag 2 hubs on flow? Thanks

This is only useful if you need different input settings for specific nodes. Then you need one hub with Process and output and one hub with Process and output on state change.

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.

3 participants