Bulk update data type is incorrect in swagger spec #7229

Closed
opened 2025-12-29 20:20:41 +01:00 by adam · 3 comments
Owner

Originally created by @abhi1693 on GitHub (Nov 15, 2022).

Originally assigned to: @abhi1693 on GitHub.

NetBox version

v3.3.7

Python version

3.10

Steps to Reproduce

As per the swagger doc, the bulk payload requires a dict
image

However, when trying to run the same, it throws an error

image

The issue is seen for all models in the spec for PUT and PATCH using the bulk operations

Expected Behavior

The spec should display that the required payload should be a List of items/dictionaries

Observed Behavior

The spec displays it needs a dictionary

Originally created by @abhi1693 on GitHub (Nov 15, 2022). Originally assigned to: @abhi1693 on GitHub. ### NetBox version v3.3.7 ### Python version 3.10 ### Steps to Reproduce As per the swagger doc, the bulk payload requires a dict ![image](https://user-images.githubusercontent.com/5083532/201976485-b0f1178f-66ec-487d-9fac-7c1ed3936804.png) However, when trying to run the same, it throws an error ![image](https://user-images.githubusercontent.com/5083532/201976682-21121789-fe6e-49a8-bc3d-0576a8696c14.png) The issue is seen for all models in the spec for PUT and PATCH using the bulk operations ### Expected Behavior The spec should display that the required payload should be a List of items/dictionaries ### Observed Behavior The spec displays it needs a dictionary
adam added the type: bugstatus: accepted labels 2025-12-29 20:20:41 +01:00
adam closed this issue 2025-12-29 20:20:41 +01:00
Author
Owner

@abhi1693 commented on GitHub (Jan 12, 2023):

@jeremystretch @arthanson @kkthxbye-code I'd like to take this

@abhi1693 commented on GitHub (Jan 12, 2023): @jeremystretch @arthanson @kkthxbye-code I'd like to take this
Author
Owner

@jeremystretch commented on GitHub (Jan 12, 2023):

Thanks @abhi1693!

@jeremystretch commented on GitHub (Jan 12, 2023): Thanks @abhi1693!
Author
Owner

@jeremystretch commented on GitHub (Apr 10, 2023):

We discovered during review of PR #11481 that resolving this adequately will likely require a significant amount of further work. Given the upcoming release of NetBox v3.5, which replaces the current schema with OpenAPI 3.0 (see #9608), we're going to abandon this change.

@jeremystretch commented on GitHub (Apr 10, 2023): We discovered during review of PR #11481 that resolving this adequately will likely require a significant amount of further work. Given the upcoming release of NetBox v3.5, which replaces the current schema with OpenAPI 3.0 (see #9608), we're going to abandon this change.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#7229