Skip to content
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

Add granular permissions for navigation/menu bar #188

Open
tom0010 opened this issue Dec 5, 2024 · 0 comments
Open

Add granular permissions for navigation/menu bar #188

tom0010 opened this issue Dec 5, 2024 · 0 comments
Labels
type: feature A new feature or functionality

Comments

@tom0010
Copy link

tom0010 commented Dec 5, 2024

Plugin Version

0.5.1

Proposed functionality

When adding permissions for a user to not allow branching capability, the app/plugin still shows on the navigation bar. If you click "Branches" or "Change Diffs" you get a friendly error page saying: "You do not have permission to access this page."

I am proposing to add extra permissions to navigation.py so when a user does not have permission to see the branching plugin, it will be hidden.

If accepted, I am happy to write a PR for this.

Use case

This offers better UX, users will not see the plugin, and it will look a lot cleaner overall.
There is no point in showing entries in the navigation bar when they cannot be accessed.

External dependencies

No response

@tom0010 tom0010 added the type: feature A new feature or functionality label Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature A new feature or functionality
Projects
None yet
Development

No branches or pull requests

1 participant