The VirtualMachineType in GraphQL is missing the contacts property #9319

Closed
opened 2025-12-29 20:48:24 +01:00 by adam · 4 comments
Owner

Originally created by @klstilling on GitHub (Mar 5, 2024).

Originally assigned to: @jeremystretch on GitHub.

NetBox version

v3.7.3

Feature type

Data model extension

Proposed functionality

I propose that the ContactMixin class is added to the VirtualMachineType class in the GraphQL files. I was wondering where I could find it, as I could find the property in the DeviceType class. I have a pull request ready which I'd be happy to submit for review.

Use case

It allows users to get information about contacts associated with virtual machine objects

Database changes

None

External dependencies

None

Originally created by @klstilling on GitHub (Mar 5, 2024). Originally assigned to: @jeremystretch on GitHub. ### NetBox version v3.7.3 ### Feature type Data model extension ### Proposed functionality I propose that the `ContactMixin` class is added to the `VirtualMachineType` class in the GraphQL files. I was wondering where I could find it, as I could find the property in the `DeviceType` class. I have a pull request ready which I'd be happy to submit for review. ### Use case It allows users to get information about contacts associated with virtual machine objects ### Database changes None ### External dependencies None
adam added the type: bugstatus: acceptedtopic: GraphQLseverity: medium labels 2025-12-29 20:48:24 +01:00
adam closed this issue 2025-12-29 20:48:24 +01:00
Author
Owner

@jeremystretch commented on GitHub (Mar 8, 2024):

Sounds like a bug.

@jeremystretch commented on GitHub (Mar 8, 2024): Sounds like a bug.
Author
Owner

@klstilling commented on GitHub (Mar 10, 2024):

@jeremystretch I'd be happy to help. By applying the changes in e76b0184a7 the property becomes available in the GraphQL API. If there's anything I can do, just poke me.

@klstilling commented on GitHub (Mar 10, 2024): @jeremystretch I'd be happy to help. By applying the changes in https://github.com/klstilling/netbox/commit/e76b0184a7235209a36fe49e0d6a2c47043a2d57 the property becomes available in the GraphQL API. If there's anything I can do, just poke me.
Author
Owner

@arthanson commented on GitHub (Mar 11, 2024):

This is blocked by #9856 as it is completely rewritten what is returned.

@arthanson commented on GitHub (Mar 11, 2024): This is blocked by #9856 as it is completely rewritten what is returned.
Author
Owner

@jeremystretch commented on GitHub (Mar 13, 2024):

Although we are completely gutting the GraphQL code with the move to Strawberry in v4.0, this seems like a simple fix we can squeeze into the current code base without much hassle.

@jeremystretch commented on GitHub (Mar 13, 2024): Although we are completely gutting the GraphQL code with the move to Strawberry in v4.0, this seems like a simple fix we can squeeze into the current code base without much hassle.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#9319