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

Removal of staff_only=True from Admin menu items #15754

Closed
markkuleinio opened this issue Apr 17, 2024 · 1 comment
Closed

Removal of staff_only=True from Admin menu items #15754

markkuleinio opened this issue Apr 17, 2024 · 1 comment
Assignees
Labels
status: accepted This issue has been accepted for implementation type: feature Introduction of new functionality to the application
Milestone

Comments

@markkuleinio
Copy link
Contributor

NetBox version

v4.0-beta1

Feature type

Change to existing functionality

Proposed functionality

The proposal is to remove staff_only=True condition from Admin menu items.

Some additional changes will be required to ensure that a normal user (without Superuser access) does not see the Admin menu items where they don't have access anyway.

Use case

Currently, if a Superuser has the Staff status removed, the user cannot access the Admin menu items anymore.

Discussed in #15733. Related comments also in the mainteners meeting minutes in #15089

Database changes

None

External dependencies

None

@markkuleinio markkuleinio added status: needs triage This issue is awaiting triage by a maintainer type: feature Introduction of new functionality to the application labels Apr 17, 2024
@jeremystretch jeremystretch added status: under review Further discussion is needed to determine this issue's scope and/or implementation and removed status: needs triage This issue is awaiting triage by a maintainer labels Apr 17, 2024
@jeremystretch
Copy link
Member

Some additional context:

  • In NetBox v3.6, the UI views for managing users, groups, permissions, API tokens, and configuration revisions were moved from the "backend" Django admin UI to the primary UI.
  • In NetBox v4.0, the remaining admin views were relocated to the primary UI, and the Django admin UI is disabled by default. (It can still be enabled for plugins which require it.)
  • A custom user model was introduced in NetBox v4.0.

As our ultimate goal is to remove the Django admin UI from NetBox entirely, and as we now have the ability to customize fields on the user model, IMO we should drop the is_staff field entirely and rely on the permissions framework to control the display of individual navigation menu items.

@jeremystretch jeremystretch self-assigned this Apr 19, 2024
@jeremystretch jeremystretch added status: accepted This issue has been accepted for implementation and removed status: under review Further discussion is needed to determine this issue's scope and/or implementation labels Apr 19, 2024
@jeremystretch jeremystretch added this to the v4.0 milestone Apr 19, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: accepted This issue has been accepted for implementation type: feature Introduction of new functionality to the application
Projects
None yet
Development

No branches or pull requests

3 participants