Add LC/APC, SC/APC and LSH/APC port types #2377

Closed
opened 2025-12-29 17:25:30 +01:00 by adam · 2 comments
Owner

Originally created by @gdelafond on GitHub (Feb 14, 2019).

Environment

  • Python version: 3.5.3
  • NetBox version: 2.5.6

Proposed Functionality

Add LC/APC, SC/APC and LSH/APC port types.

Use Case

We need to track LC PC/APC and SC PC/APC port type on patch panel.

Database Changes

None.

External Dependencies

None.

Originally created by @gdelafond on GitHub (Feb 14, 2019). ### Environment * Python version: <!-- Example: 3.5.4 --> 3.5.3 * NetBox version: <!-- Example: 2.3.6 --> 2.5.6 ### Proposed Functionality Add LC/APC, SC/APC and LSH/APC port types. ### Use Case We need to track LC PC/APC and SC PC/APC port type on patch panel. <!-- Note any changes to the database schema necessary to support the new feature. For example, does the proposal require adding a new model or field? (Not all new features require database changes.) ---> ### Database Changes None. <!-- List any new dependencies on external libraries or services that this new feature would introduce. For example, does the proposal require the installation of a new Python package? (Not all new features introduce new dependencies.) --> ### External Dependencies None.
adam added the status: acceptedtype: feature labels 2025-12-29 17:25:30 +01:00
adam closed this issue 2025-12-29 17:25:30 +01:00
Author
Owner

@jeremystretch commented on GitHub (Feb 15, 2019):

Please revise your request to specify the exact types you're proposing and include justification for why the existing types aren't sufficient.

@jeremystretch commented on GitHub (Feb 15, 2019): Please revise your request to specify the _exact_ types you're proposing and include justification for why the existing types aren't sufficient.
Author
Owner

@gdelafond commented on GitHub (Feb 15, 2019):

We have patch panel with 4 differents port types: LC/PC, LC/APC, SC/PC and SC/APC.

Currently in Netbox there is already LC and SC port types. LC is usually a shortcut for LC/PC and SC for SC/PC. That's why I want to add LC/APC and SC/APC.

As I know that LSH/APC exists too (cf #2569), I have included it in my PR.

So this request and the PR is to add the following port types :

  • LC/APC
  • SC/APC
  • LSH/APC

If you want, in addition I can rename existing LC to LC/PC and existing SC to LC/PC.

@gdelafond commented on GitHub (Feb 15, 2019): We have patch panel with 4 differents port types: LC/PC, LC/APC, SC/PC and SC/APC. Currently in Netbox there is already LC and SC port types. LC is usually a shortcut for LC/PC and SC for SC/PC. That's why I want to add LC/APC and SC/APC. As I know that LSH/APC exists too (cf #2569), I have included it in my PR. So this request and the PR is to add the following port types : - LC/APC - SC/APC - LSH/APC If you want, in addition I can rename existing LC to LC/PC and existing SC to LC/PC.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#2377