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

Reset disarm reason at moment of arming #3360

Merged
merged 1 commit into from Jun 15, 2018

Conversation

Projects
None yet
1 participant
@digitalentity
Copy link
Member

commented Jun 13, 2018

Currently we only update disarm reason if disarm() is called correctly, however disarm may happen as an accidental reset of ARMED flag but then disarm reason won't be helpful in debugging this.

@digitalentity digitalentity added the BUG label Jun 13, 2018

@digitalentity digitalentity added this to the 2.0 milestone Jun 13, 2018

@digitalentity digitalentity merged commit c4c1b42 into development Jun 15, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@digitalentity digitalentity deleted the de_reset_disarm_reason branch Jun 15, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.