Skip to content

Conversation

janjurca
Copy link

Hello
I've made change that is maybe worth of discussion. The actual SensorInterrupt does send it's value only when the interrupt happens. But when the transmission fails for some reason then the value is not propagated to server not even on report interval.

So I've added the onLoop() method to SensorInterrupt.h.

@user2684
Copy link
Contributor

Hi! A few comments:

  • What if the transmission was successful, is a second transmission taking place anyway?
  • Any risk for non-sleeping node that loop is executed continuously?
  • Ensure the PR is against development branch not master
    Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants