[PR #6449] [MERGED] Fixed #6438 Device Type Import does not import/export description/label fields for many components #13128

Closed
opened 2025-12-29 22:25:34 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbox-community/netbox/pull/6449
Author: @991jo
Created: 5/20/2021
Status: Merged
Merged: 5/21/2021
Merged by: @jeremystretch

Base: developHead: device_type_import_fix


📝 Commits (1)

  • ea6c8a1 Fixed #6438 Device Type Import does not import/export description/label fields for many components

📊 Changes

3 files changed (+25 additions, -8 deletions)

View changed files

📝 docs/release-notes/version-2.11.md (+1 -0)
📝 netbox/dcim/forms.py (+8 -8)
📝 netbox/dcim/models/devices.py (+16 -0)

📄 Description

Fixes: #6438

Some Components of Device Types did not import and export their label and/or description fields.

I fixed it by adding the fields to the appropriate lists where needed.

A less redundant solution could be to only have the list of fields in the ImportForms and iterate over the Meta.fields attribute of the ImportForms in the export function.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/netbox-community/netbox/pull/6449 **Author:** [@991jo](https://github.com/991jo) **Created:** 5/20/2021 **Status:** ✅ Merged **Merged:** 5/21/2021 **Merged by:** [@jeremystretch](https://github.com/jeremystretch) **Base:** `develop` ← **Head:** `device_type_import_fix` --- ### 📝 Commits (1) - [`ea6c8a1`](https://github.com/netbox-community/netbox/commit/ea6c8a1a65f2db8a6ff6937a83fb90726954ec5e) Fixed #6438 Device Type Import does not import/export description/label fields for many components ### 📊 Changes **3 files changed** (+25 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `docs/release-notes/version-2.11.md` (+1 -0) 📝 `netbox/dcim/forms.py` (+8 -8) 📝 `netbox/dcim/models/devices.py` (+16 -0) </details> ### 📄 Description <!-- Thank you for your interest in contributing to NetBox! Please note that our contribution policy requires that a feature request or bug report be opened for approval prior to filing a pull request. This helps avoid wasting time and effort on something that we might not be able to accept. Please indicate the relevant feature request or bug report below. IF YOUR PULL REQUEST DOES NOT REFERENCE AN ACCEPTED BUG REPORT OR FEATURE REQUEST, IT WILL BE MARKED AS INVALID AND CLOSED. --> ### Fixes: #6438 <!-- Please include a summary of the proposed changes below. --> Some Components of Device Types did not import and export their label and/or description fields. I fixed it by adding the fields to the appropriate lists where needed. A less redundant solution could be to only have the list of fields in the ImportForms and iterate over the Meta.fields attribute of the ImportForms in the export function. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 22:25:34 +01:00
adam closed this issue 2025-12-29 22:25:34 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#13128