fix: Stop service modal form disabling services for read-only admin #4994

Merged
merged 1 commit into from Nov 16, 2016

Projects

None yet

4 participants

@laf
Member
laf commented Nov 15, 2016

Please note

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

Fixes: #4945

@laf laf fix: Stop service modal form disabling services for read-only admin
946ac72
@laf laf added Bug WebUI labels Nov 15, 2016
@scrutinizer-notifier

The inspection completed: No new issues

@murrant
Contributor
murrant commented Nov 16, 2016

Looks good, but the buttons should be disabled or removed for non-admin users.

@murrant murrant merged commit 1acd938 into librenms:master Nov 16, 2016

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@laf laf deleted the laf:issue-4945 branch Nov 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment