Remove Circuit Swap Terminations #11778

Closed
opened 2025-12-29 21:49:45 +01:00 by adam · 1 comment
Owner

Originally created by @arthanson on GitHub (Oct 24, 2025).

Originally assigned to: @arthanson on GitHub.

Proposed Changes

Within Circuits detail view there is a "Swap" button on the terminations that will switch the places of the terminations.

Justification

This is a niche utility function that can be replicated by simply editing the first termination to unset the termination side, then editing the second termination and setting its termination side. Several points on this:

  • It doesn't add any new functionality, just a convenience function
  • It is a one-off there is no consistency with cables or such (they don't have a swap function).
  • There is no corresponding REST API that does the same thing.
  • It adds complexity to the code.
  • The way it does the swap makes it difficult for branching and requires maintaining / debugging unique code. See #282 in branching
Originally created by @arthanson on GitHub (Oct 24, 2025). Originally assigned to: @arthanson on GitHub. ### Proposed Changes Within Circuits detail view there is a "Swap" button on the terminations that will switch the places of the terminations. ### Justification This is a niche utility function that can be replicated by simply editing the first termination to unset the termination side, then editing the second termination and setting its termination side. Several points on this: * It doesn't add any new functionality, just a convenience function * It is a one-off there is no consistency with cables or such (they don't have a swap function). * There is no corresponding REST API that does the same thing. * It adds complexity to the code. * The way it does the swap makes it difficult for branching and requires maintaining / debugging unique code. See [#282](https://github.com/netboxlabs/netbox-branching/issues/282) in branching
adam added the type: deprecationnetbox labels 2025-12-29 21:49:45 +01:00
adam closed this issue 2025-12-29 21:49:45 +01:00
Author
Owner

@arthanson commented on GitHub (Oct 28, 2025):

merged in #20684. Closing

@arthanson commented on GitHub (Oct 28, 2025): merged in [#20684](https://github.com/netbox-community/netbox/pull/20684). Closing
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#11778