Add ping and RxLevel for SAF devices #4840

Merged
merged 5 commits into from Oct 21, 2016

Projects

None yet

4 participants

@barryodonovan
Contributor

Added ping and RxLevel graphs to SAF device page

Please note

Please read this information carefully. You can run ./scripts/pre-commit.php to check your code before submitting.

barryodonovan added some commits Oct 20, 2016
barryodonovan I agree to the conditions of the Contributor Agreement contained in d…
…oc/General/Contributing.md.
e525da0
barryodonovan Merge branch 'contrib-agree'
Conflicts:
	AUTHORS.md
eca129b
barryodonovan Added pig and RxLevel graphs to device dahboard page
cf1c8f5
@laf laf changed the title from Issue#4839 to Add ping and RxLevel for SAF devices Oct 20, 2016
@laf
Member
laf commented Oct 20, 2016

Your update of the AUTHORS file contains some patch code, please remove this.

@laf laf added the Blocker label Oct 20, 2016
@barryodonovan
Contributor

I've edited AUTHORS but not sure if OK.
Might be easier for me to cancel this PR and try again?
I think i messed up with my base for the branch for this PR

@laf laf rebased
58e75b5
@laf laf removed the Blocker label Oct 21, 2016
@scrutinizer-notifier

The inspection completed: No new issues

@laf laf merged commit 4846538 into librenms:master Oct 21, 2016

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment