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.

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

Add "site" column to power feed list #13743

Closed
pv2b opened this issue Sep 12, 2023 · 1 comment · Fixed by #14214
Closed

Add "site" column to power feed list #13743

pv2b opened this issue Sep 12, 2023 · 1 comment · Fixed by #14214
Assignees
Labels
status: accepted This issue has been accepted for implementation type: feature Introduction of new functionality to the application

Comments

@pv2b
Copy link
Contributor

pv2b commented Sep 12, 2023

NetBox version

v3.6.1

Feature type

Change to existing functionality

Proposed functionality

I propose adding "Site" as a selectable column for the power feed view. The power feed model doesn't directly have a "Site" field, so the "Site" column would be populated from the associated Rack.

Use case

When looking at a list of power feeds, it's helpful to, at a glance, know what site the power feed belongs to. This is especially useful if you have duplicate rack names across sites.

Database changes

None.

External dependencies

None.

@pv2b pv2b added the type: feature Introduction of new functionality to the application label Sep 12, 2023
@pv2b
Copy link
Contributor Author

pv2b commented Sep 12, 2023

I just realized I can acheive this with a Custom Link, so that's a workaround for now.

If anyone else needs this, just import the following Custom Link using this YAML in the bulk importer:

---
name: Site
content_types: dcim.powerfeed
weight: 1000
button_class: ghost-dark
link_text: '{{ object.rack.site.name }}'
link_url: '{{ object.rack.site.url }}'
enabled: true

Once the Custom Link is created, it can then be used as a field in the table view (which is essentialy what the FR here is asking for.)

The downside to this is that you'll get an extra link on the power feed view to the site, but... I guess that might be considered useful?

Anyway, I still think it should be in Netbox from the start but given it's easy enough to customize Netbox and get the same result this probably doesn't need to be prioritized.

@jeremystretch jeremystretch added the status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation label Nov 6, 2023
@abhi1693 abhi1693 self-assigned this Nov 8, 2023
@abhi1693 abhi1693 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 Nov 8, 2023
abhi1693 added a commit that referenced this issue Nov 8, 2023
abhi1693 added a commit that referenced this issue Nov 8, 2023
jeremystretch pushed a commit that referenced this issue Nov 9, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: accepted This issue has been accepted for implementation type: feature Introduction of new functionality to the application
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants