Fixes a number of user permission issues #1411

Merged
merged 5 commits into from Jul 10, 2015

Projects

None yet

4 participants

@laf
Member
laf commented Jul 7, 2015

Fixes #1289

@laf laf added Bug WebUI labels Jul 7, 2015
@paulgear
Member

This seems pretty good to me, but there's a lot of duplication inside each of those diffs that I'd really like to see consolidated into a simple common function. Thoughts?

@f0o
Member
f0o commented Jul 10, 2015

I share @paulgear's opinions

@laf
Member
laf commented Jul 10, 2015

The queries do vary slightly but feel free to fix this after it's merged as this fixes a current bug.

@f0o f0o merged commit 7560d5d into librenms:master Jul 10, 2015

2 checks passed

Auto-Deploy Build finished.
Details
Scrutinizer No new issues
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment