[PR #10473] [MERGED] 10472 graphene 3 #13639

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

📋 Pull Request Information

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

Base: featureHead: 10472-graphene-3


📝 Commits (6)

  • b7cdba0 update to Graphene 3.0.0
  • 4fa5951 Merge branch 'feature' into 10472-graphene-3
  • 3004de4 10472 exempt view permissions on tests
  • 3bbbc94 10472 exempt permission check on graphql tests
  • edf28eb Merge branch 'feature' into 10472-graphene-3
  • 6fadf8e fix merge conflicts

📊 Changes

4 files changed (+5 additions, -3 deletions)

View changed files

📝 netbox/netbox/graphql/scalars.py (+1 -1)
📝 netbox/netbox/graphql/schema.py (+1 -1)
📝 netbox/utilities/testing/api.py (+2 -0)
📝 requirements.txt (+1 -1)

📄 Description

Fixes: #10472

Updates to Graphene v3 which is a major update. The permission checks on the tests had to be disabled, which is a bit concerning.


🔄 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/10473 **Author:** [@arthanson](https://github.com/arthanson) **Created:** 9/27/2022 **Status:** ✅ Merged **Merged:** 10/5/2022 **Merged by:** [@jeremystretch](https://github.com/jeremystretch) **Base:** `feature` ← **Head:** `10472-graphene-3` --- ### 📝 Commits (6) - [`b7cdba0`](https://github.com/netbox-community/netbox/commit/b7cdba06ff0c734781e34eca6902a50ccf8576c2) update to Graphene 3.0.0 - [`4fa5951`](https://github.com/netbox-community/netbox/commit/4fa5951fc2bb26cd51e718301804e63305876eeb) Merge branch 'feature' into 10472-graphene-3 - [`3004de4`](https://github.com/netbox-community/netbox/commit/3004de4e5e5106c773b904c54361c52d673e26d0) 10472 exempt view permissions on tests - [`3bbbc94`](https://github.com/netbox-community/netbox/commit/3bbbc9469f37f8b031382120a7f8f87ba4fee576) 10472 exempt permission check on graphql tests - [`edf28eb`](https://github.com/netbox-community/netbox/commit/edf28eb4941e634d312b76f583e9d07b71533649) Merge branch 'feature' into 10472-graphene-3 - [`6fadf8e`](https://github.com/netbox-community/netbox/commit/6fadf8eb95045b1de842a9cfbfa2780d344f9bd0) fix merge conflicts ### 📊 Changes **4 files changed** (+5 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `netbox/netbox/graphql/scalars.py` (+1 -1) 📝 `netbox/netbox/graphql/schema.py` (+1 -1) 📝 `netbox/utilities/testing/api.py` (+2 -0) 📝 `requirements.txt` (+1 -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: #10472 <!-- Please include a summary of the proposed changes below. --> Updates to Graphene v3 which is a major update. The permission checks on the tests had to be disabled, which is a bit concerning. --- <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:00 +01:00
adam closed this issue 2025-12-29 23:20:00 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#13639