Skip to content

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.

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
@cruse1977 cruse1977 merged commit 0a14e40 into netbox-community:dev Apr 19, 2025
6 checks passed
@pheus pheus deleted the feature/255-group-navigation-items branch April 19, 2025 21:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Group Navigation Items

2 participants