-
Notifications
You must be signed in to change notification settings - Fork 0
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
Permissions best practices #23
Comments
How do we want to track these, this was the first on the list here: #5 (comment) |
I think this is a little more specific, you wanted to ensure they were there - I wanted to ensure they are correct 😄 Maybe have this one supersede yours because it is more detailed? |
It is my understanding if they are not named correctly, they will not work correctly, perhaps I misunderstand. That being said, "how do we want to track these"? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Environment
Proposed Functionality
Lint, whether permissions conform to the naming schema (see here).
Use Case
Prevent cases like this:
https://github.com/networktocode-llc/nautobot-plugin-os-upgrades/pull/50
In this case it appears like permissions are forbidding you from looking at the views but when you click the greyed out item in the nav bar you are taken to the page anyway.
The text was updated successfully, but these errors were encountered: