[PR #10574] [MERGED] 9478 link peers to GraphQL #13657

Closed
opened 2025-12-29 23:20:06 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbox-community/netbox/pull/10574
Author: @arthanson
Created: 10/5/2022
Status: Merged
Merged: 10/6/2022
Merged by: @jeremystretch

Base: featureHead: 9478-link-peer


📝 Commits (7)

📊 Changes

4 files changed (+84 additions, -6 deletions)

View changed files

netbox/dcim/graphql/gfk_mixins.py (+59 -0)
📝 netbox/dcim/graphql/mixins.py (+7 -0)
📝 netbox/netbox/graphql/types.py (+13 -5)
📝 netbox/utilities/testing/api.py (+5 -1)

📄 Description

Fixes: #9478

Adds link_peers to GraphQL, also adds class_type to all models which is very useful for any Django GFK / GraphQL Union types going forward. There are several other GFK fields that need to be added to GraphQL but will do those as separate PRs.

Note: This will probably get a merge conflict when #10556 is added.


🔄 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/10574 **Author:** [@arthanson](https://github.com/arthanson) **Created:** 10/5/2022 **Status:** ✅ Merged **Merged:** 10/6/2022 **Merged by:** [@jeremystretch](https://github.com/jeremystretch) **Base:** `feature` ← **Head:** `9478-link-peer` --- ### 📝 Commits (7) - [`38dd289`](https://github.com/netbox-community/netbox/commit/38dd28929ef39b277482f909926b778e3f40d288) 9468 add link_peer to GraphQL - [`7d976f0`](https://github.com/netbox-community/netbox/commit/7d976f04770ea192e3dea08be430e5325a78c7ff) Merge branch 'feature' into 9478-link-peer - [`5de5422`](https://github.com/netbox-community/netbox/commit/5de5422ad73a777ce4f4274a4193baf3d2f62446) 9478 add class_type - [`9c55e43`](https://github.com/netbox-community/netbox/commit/9c55e43978ca9a81d5129e77de723b5d571eb5d2) 9478 fix feature merge - [`bd2de21`](https://github.com/netbox-community/netbox/commit/bd2de21d931ae66c9319cab89bc89ca5f0ffb2f8) 9478 fix tests - [`f7c6605`](https://github.com/netbox-community/netbox/commit/f7c660572ea1b629f6d98b1a253f6fb93cf9b6d2) 9478 fix tests - [`94c5f7f`](https://github.com/netbox-community/netbox/commit/94c5f7faf02b3e830b5ee25125d076ba367957d4) 9478 fix tests ### 📊 Changes **4 files changed** (+84 additions, -6 deletions) <details> <summary>View changed files</summary> ➕ `netbox/dcim/graphql/gfk_mixins.py` (+59 -0) 📝 `netbox/dcim/graphql/mixins.py` (+7 -0) 📝 `netbox/netbox/graphql/types.py` (+13 -5) 📝 `netbox/utilities/testing/api.py` (+5 -1) </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 approved and assigned prior to filing a pull request. This helps avoid wasting time and effort on something that we might not be able to accept. IF YOUR PULL REQUEST DOES NOT REFERENCE AN ISSUE WHICH HAS BEEN ASSIGNED TO YOU, IT WE BE CLOSED AUTOMATICALLY. Specify your assigned issue number on the line below. --> ### Fixes: #9478 <!-- Please include a summary of the proposed changes below. --> Adds link_peers to GraphQL, also adds class_type to all models which is very useful for any Django GFK / GraphQL Union types going forward. There are several other GFK fields that need to be added to GraphQL but will do those as separate PRs. **Note**: This will probably get a merge conflict when #10556 is added. --- <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 23:20:06 +01:00
adam closed this issue 2025-12-29 23:20:07 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#13657