Skip to content

refactor(navigation): Group plugin menu items #256

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

refactor(navigation): Group plugin menu items #256

wants to merge 1 commit into from

Conversation

pheus
Copy link
Contributor

@pheus pheus commented Mar 23, 2025

Pull Request

Related Issue

Fixes: #255 [Feature] Group Navigation Items

New Behavior

  • Groups navigation items under Access Lists, Rules, and Assignments for improved readability and maintainability.
  • Splits menu definitions into distinct variables for better clarity.

Contrast to Current Behavior

  • Previously, navigation items were listed without logical grouping.
  • This refactor enhances structure but introduces no functional changes.

Discussion: Benefits and Drawbacks

  • Improves navigation efficiency, making it easier to locate relevant items.
  • Enhances scalability, ensuring a well-organized menu as new models are introduced.
  • Primarily a cosmetic improvement, but contributes to a better user experience.

Changes to the Documentation

None required.

Proposed Release Note Entry

  • Reorganized the plugin menu, grouping items under Access Lists, Rules, and Assignments.

Double Check

  • I have explained my PR according to the information in the comments
    or in a linked issue.
  • My PR targets the dev branch.

@pheus
refactor(navigation): Group plugin menu items
bc3de78 
Reorganizes the plugin menu by grouping items under Access Lists, Rules,
and Assignments for improved readability and maintainability.
Splits menu definitions into distinct variables for clarity.
No functional changes introduced.

Fixes: netbox-community#255
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feature]: Group Navigation Items
1 participant
@pheus