-
Notifications
You must be signed in to change notification settings - Fork 282
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
Display: Allow limited view access to non-destructive edit-only features #2718
Display: Allow limited view access to non-destructive edit-only features #2718
Conversation
96356df
to
6b770fc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a different approach we can take here, please can we review this one together.
6b770fc
to
e043fee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much more inline with the intension! great.
I have one comment regarding the display group access
TEST: /Administration END |
Test Suite: failed ❌ |
TEST: /Administration END |
Test Suite: succeeded ✅ |
Changes
Relates to: xibosignage/xibo#3479