Skip to content
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

Device type power outlet component power port dropdown #17636

Closed
ChrisRecioHive opened this issue Sep 27, 2024 · 1 comment · Fixed by #17690
Closed

Device type power outlet component power port dropdown #17636

ChrisRecioHive opened this issue Sep 27, 2024 · 1 comment · Fixed by #17690
Assignees
@DanSheps
Labels
severity: low Does not significantly disrupt application functionality, or a workaround is available status: accepted This issue has been accepted for implementation type: bug A confirmed report of unexpected behavior in the application

Comments

@ChrisRecioHive
Copy link

Deployment Type

Self-hosted

NetBox Version

v4.1.1

Python Version

3.11

Steps to Reproduce

  1. Open a device type
  2. Add a power outlet component
  3. Open the "Power port" dropdown menu

Expected Behavior

The power port dropdown menu would display only the power ports associated with that device type.
Ex: Creating a PDU device type. Associate power outlets with the power port on the PDU

Observed Behavior

The power port dropdown menu displays every device types power ports.
image
@ChrisRecioHive ChrisRecioHive added status: needs triage This issue is awaiting triage by a maintainer type: bug A confirmed report of unexpected behavior in the application labels Sep 27, 2024
@arthanson arthanson added status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation severity: low Does not significantly disrupt application functionality, or a workaround is available and removed status: needs triage This issue is awaiting triage by a maintainer labels Sep 30, 2024
@wksaminieminen
Copy link

wksaminieminen commented Oct 4, 2024 •
edited
Loading

More details:
After creating a device from a template which has no power port connected, the filtering works.
But filtering does not work on Device Types.

This works in 4.0.8, but does not work on 4.1.x or later.

Screenshot from 4.0.8 self hosted device template:
No issue on 4.0.8

Snapshot from 4.1.1 self hosted device template:
Issue on 4.1.1

Screenshot from 4.1.1 self hosted device view:
No issue on 4.1.1 device view:

DanSheps added a commit that referenced this issue Oct 7, 2024
@DanSheps
Fixes: #17636 - Correct typo in Power Outlet Template form for Power …
500521c 
…Port field
@DanSheps DanSheps mentioned this issue Oct 7, 2024
Merged
@DanSheps DanSheps self-assigned this Oct 7, 2024
@DanSheps DanSheps added status: accepted This issue has been accepted for implementation and removed status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation labels Oct 7, 2024
jeremystretch pushed a commit that referenced this issue Oct 9, 2024
@DanSheps @jeremystretch
Fixes: #17636 - Correct typo in Power Outlet Template form for Power …
23e6534 
…Port field
jeremystretch added a commit that referenced this issue Oct 9, 2024
@jeremystretch
Changelog for #17216, #17562, #17636
8e636c5
bctiemann pushed a commit that referenced this issue Oct 11, 2024
@DanSheps @bctiemann
Fixes: #17636 - Correct typo in Power Outlet Template form for Power …
0944284 
…Port field
bctiemann pushed a commit that referenced this issue Oct 11, 2024
@jeremystretch @bctiemann
Changelog for #17216, #17562, #17636
864acf5
jeremystretch pushed a commit to alehaa/netbox that referenced this issue Oct 11, 2024
@DanSheps @jeremystretch
Fixes: netbox-community#17636 - Correct typo in Power Outlet Template…
3382d96 
… form for Power Port field
jeremystretch added a commit to alehaa/netbox that referenced this issue Oct 11, 2024
@jeremystretch
Changelog for netbox-community#17216, netbox-community#17562, netbox-…
ef49ecb 
…community#17636
@jeremystretch jeremystretch mentioned this issue Oct 15, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DanSheps DanSheps

Labels
severity: low Does not significantly disrupt application functionality, or a workaround is available status: accepted This issue has been accepted for implementation type: bug A confirmed report of unexpected behavior in the application
Projects
None yet
Milestone
No milestone
4 participants
@arthanson @DanSheps @ChrisRecioHive @wksaminieminen