starred/netbox
1
0
Fork 0
mirror of https://github.com/netbox-community/netbox.git synced 2026-01-11 21:10:29 +01:00
Code Issues 273 Packages Projects Releases 10 Wiki Activity

Searching for a Cross-Connect ID links to the Site instead to the circuit-termination #7632

New Issue
Closed
opened 2025-12-29 20:26:15 +01:00 by adam · 6 comments
adam commented 2025-12-29 20:26:15 +01:00
Owner
Copy Link

Originally created by @lairsdragon on GitHub (Feb 10, 2023).

Originally assigned to: @jeremystretch on GitHub.

NetBox version

v3.4.4

Python version

3.10

Steps to Reproduce

  • Add a Cross-Connect ID to a Cirtuit Termination
  • Search for the ID
  • Inspect the Link Target under Object

cross-connect-search

Expected Behavior

Under Object I expect a Link back to the Circuit
circuit

Observed Behavior

Under Object a Link to the Site is generated

Originally created by @lairsdragon on GitHub (Feb 10, 2023). Originally assigned to: @jeremystretch on GitHub. ### NetBox version v3.4.4 ### Python version 3.10 ### Steps to Reproduce - Add a Cross-Connect ID to a Cirtuit Termination - Search for the ID - Inspect the Link Target under Object ![cross-connect-search](https://user-images.githubusercontent.com/1344232/218131484-0f867bf6-f56a-4e42-a67f-225df046a5c8.png) ### Expected Behavior Under Object I expect a Link back to the Circuit ![circuit](https://user-images.githubusercontent.com/1344232/218131491-0ae74481-e02c-4045-801f-8ee30a221f45.png) ### Observed Behavior Under Object a Link to the Site is generated
adam added the type: bugstatus: accepted labels 2025-12-29 20:26:15 +01:00
adam closed this issue 2025-12-29 20:26:15 +01:00
adam commented 2025-12-29 20:26:16 +01:00
Author
Owner
Copy Link

@kkthxbye-code commented on GitHub (Feb 10, 2023):

Under Object I expect a Link back to the corresponding Circuit Termination

There's no details nor list pages for CircuitTerminations. Can you provide an example URL of what you want it to link to?

In any case the current functionality is intended, so you probably need a feature request if you want to suggest that it should be changed.

@kkthxbye-code commented on GitHub (Feb 10, 2023): > Under Object I expect a Link back to the corresponding Circuit Termination There's no details nor list pages for CircuitTerminations. Can you provide an example URL of what you want it to link to? In any case the current functionality is intended, so you probably need a feature request if you want to suggest that it should be changed.
adam commented 2025-12-29 20:26:16 +01:00
Author
Owner
Copy Link

@lairsdragon commented on GitHub (Feb 10, 2023):

I missed that there is not detail page for Cirtciut Termination.

A link back to the matching Circuit would be helpful. At the moment I have to visit all Circuits of a site and that way its not better than what I could have done without the search.

I have edited the initial command and edited screenshots from a test in the demo site.

@lairsdragon commented on GitHub (Feb 10, 2023): I missed that there is not detail page for Cirtciut Termination. A link back to the matching Circuit would be helpful. At the moment I have to visit all Circuits of a site and that way its not better than what I could have done without the search. I have edited the initial command and edited screenshots from a test in the demo site.
adam commented 2025-12-29 20:26:16 +01:00
Author
Owner
Copy Link

@jsenecal commented on GitHub (Feb 10, 2023):

So technically this is pretty simple and makes sense as we only need to change from which related object the url is generated in here:
https://github.com/netbox-community/netbox/blob/develop/netbox/circuits/models/circuits.py#L201

Not sure how this would affect current users workflows however... This changes affects other views, such as the Circuit Table Side A/Z Links.

As this is technically not a bug, I would suggest that you open a Feature Request instead and describe our conclusions in there.

@jsenecal commented on GitHub (Feb 10, 2023): So technically this is pretty simple and makes sense as we only need to change from which related object the url is generated in here: https://github.com/netbox-community/netbox/blob/develop/netbox/circuits/models/circuits.py#L201 Not sure how this would affect current users workflows however... This changes affects other views, such as the Circuit Table Side A/Z Links. As this is technically not a bug, I would suggest that you open a Feature Request instead and describe our conclusions in there.
adam commented 2025-12-29 20:26:17 +01:00
Author
Owner
Copy Link

@PieterL75 commented on GitHub (Feb 13, 2023):

This is a common thing.. the relation of a circuit is to the site.. Clicking that link always brings you to the Site object.
It would be more intuitive that clicking a A/Z side brings you to the Circuit page in stead of the 'Site'

@PieterL75 commented on GitHub (Feb 13, 2023): This is a common thing.. the relation of a circuit is to the site.. Clicking that link always brings you to the Site object. It would be more intuitive that clicking a A/Z side brings you to the Circuit page in stead of the 'Site'
adam commented 2025-12-29 20:26:17 +01:00
Author
Owner
Copy Link

@jsenecal commented on GitHub (Feb 15, 2023):

@PieterL75 Circuit Table Side A/Z Links to a Provider Network if the circuit termination is of the other kind. What's your thought on that?

@jsenecal commented on GitHub (Feb 15, 2023): @PieterL75 Circuit Table Side A/Z Links to a `Provider Network` if the circuit termination is of the other kind. What's your thought on that?
adam commented 2025-12-29 20:26:17 +01:00
Author
Owner
Copy Link

@jeremystretch commented on GitHub (Feb 18, 2023):

Seems like the simplest solution is to change the default representation of a CircuitTermination to "$circuit: Termination A/Z" and have it link to its parent circuit. This should only impact cases where circuit terminations are shown independently (e.g. in search results or the change log); it won't affect the circuits list, which shows the A and Z terminations for each circuit.

@jeremystretch commented on GitHub (Feb 18, 2023): Seems like the simplest solution is to change the default representation of a CircuitTermination to "$circuit: Termination A/Z" and have it link to its parent circuit. This should only impact cases where circuit terminations are shown independently (e.g. in search results or the change log); it won't affect the circuits list, which shows the A and Z terminations for each circuit.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
beta
breaking change
complexity: high
complexity: low
complexity: medium
needs milestone
netbox
pending closure
plugin candidate
pull-request
Mirrored from GitHub Pull Request
 severity: high
severity: low
severity: medium
status: accepted
status: backlog
status: blocked
status: duplicate
status: needs owner
status: needs triage
status: revisions needed
status: under review
topic: GraphQL
topic: Internationalization
topic: OpenAPI
topic: UI/UX
topic: cabling
topic: event rules
topic: htmx navigation
topic: industrialization
topic: migrations
topic: plugins
topic: scripts
topic: templating
topic: testing
type: bug
type: deprecation
type: documentation
type: feature
type: housekeeping
type: translation
No Label status: accepted type: bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
adam
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#7632
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?