Make bulk import/update header fields case-insensitive #8617

Closed
opened 2025-12-29 20:38:58 +01:00 by adam · 2 comments
Owner

Originally created by @peteeckel on GitHub (Sep 15, 2023).

NetBox version

v3.6.1

Feature type

Change to existing functionality

Proposed functionality

Based on the brief discussion of #13773 I suggest that the handling of the header line in bulk import/update should be case insensitive.

Use case

This would avoid a couple of apparently common cases where data is silently ignored because of casing, which is often introduced when data from an export is converted to input data.

Database changes

None

External dependencies

None

Originally created by @peteeckel on GitHub (Sep 15, 2023). ### NetBox version v3.6.1 ### Feature type Change to existing functionality ### Proposed functionality Based on the brief discussion of #13773 I suggest that the handling of the header line in bulk import/update should be case insensitive. ### Use case This would avoid a couple of apparently common cases where data is silently ignored because of casing, which is often introduced when data from an export is converted to input data. ### Database changes None ### External dependencies None
adam added the type: feature label 2025-12-29 20:38:58 +01:00
adam closed this issue 2025-12-29 20:38:58 +01:00
Author
Owner

@jeremystretch commented on GitHub (Sep 15, 2023):

This would work for all built-in model fields, but it's not currently feasible for custom fields as we don't enforce case insensitivity on custom field names. For example, a model could have custom fields named foo and Foo. It's a terrible idea, of course, however it is currently a valid scenario.

@jeremystretch commented on GitHub (Sep 15, 2023): This would work for all built-in model fields, but it's not currently feasible for custom fields as we don't enforce case insensitivity on custom field names. For example, a model could have custom fields named `foo` and `Foo`. It's a terrible idea, of course, however it is currently a valid scenario.
Author
Owner

@peteeckel commented on GitHub (Sep 15, 2023):

Yuck. Well, once the PR for #11617 is implemented that won't be as much of an issue anymore either, so never mind.

@peteeckel commented on GitHub (Sep 15, 2023): Yuck. Well, once the PR for #11617 is implemented that won't be as much of an issue anymore either, so never mind.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#8617