[PR #12718] [CLOSED] DRAFT: 12552 replace mptt #14034

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

📋 Pull Request Information

Original PR: https://github.com/netbox-community/netbox/pull/12718
Author: @arthanson
Created: 5/25/2023
Status: Closed

Base: featureHead: 12552-replace-mptt


📝 Commits (10+)

  • da0c459 12552 initial removal of mptt
  • 252ca51 12552 get_descendants -> descendants, get_ancestors -> ancestors
  • c93b7cf 12552 can't have more then one ordering
  • 71a74e4 12552 test fixes
  • a480a4c 12552 add back add_related_count
  • 3bd3668 12552 add back add_related_count
  • 9ee6511 12552 remove extra param
  • ff9a9d9 12552 cleanup mptt
  • a8c9d87 12552 test fixes
  • 786394f add back mptt to requirements

📊 Changes

37 files changed (+463 additions, -134 deletions)

View changed files

📝 base_requirements.txt (+4 -0)
📝 netbox/dcim/api/nested_serializers.py (+5 -5)
📝 netbox/dcim/api/serializers.py (+2 -2)
netbox/dcim/migrations/0174_mptt_to_cte.py (+164 -0)
📝 netbox/dcim/models/device_component_templates.py (+4 -12)
📝 netbox/dcim/models/device_components.py (+5 -13)
📝 netbox/dcim/models/devices.py (+1 -1)
📝 netbox/dcim/models/sites.py (+4 -3)
📝 netbox/dcim/signals.py (+2 -2)
📝 netbox/dcim/tables/devices.py (+1 -1)
📝 netbox/dcim/tables/sites.py (+3 -3)
📝 netbox/dcim/views.py (+3 -3)
📝 netbox/extras/querysets.py (+6 -4)
📝 netbox/ipam/querysets.py (+5 -5)
📝 netbox/netbox/api/serializers/__init__.py (+2 -2)
📝 netbox/netbox/models/__init__.py (+5 -24)
📝 netbox/netbox/tables/columns.py (+4 -4)
📝 netbox/templates/dcim/site.html (+1 -1)
📝 netbox/tenancy/api/nested_serializers.py (+2 -2)
netbox/tenancy/migrations/0011_mptt_to_cte.py (+67 -0)

...and 17 more files

📄 Description

Fixes: #12552


🔄 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/12718 **Author:** [@arthanson](https://github.com/arthanson) **Created:** 5/25/2023 **Status:** ❌ Closed **Base:** `feature` ← **Head:** `12552-replace-mptt` --- ### 📝 Commits (10+) - [`da0c459`](https://github.com/netbox-community/netbox/commit/da0c459e731e0c870699929f8b40e202f48216a0) 12552 initial removal of mptt - [`252ca51`](https://github.com/netbox-community/netbox/commit/252ca516db03f8f48249bf10480ffeddcfb2cb2c) 12552 get_descendants -> descendants, get_ancestors -> ancestors - [`c93b7cf`](https://github.com/netbox-community/netbox/commit/c93b7cfc5cb583f9f2effd085cb55bc3ca876035) 12552 can't have more then one ordering - [`71a74e4`](https://github.com/netbox-community/netbox/commit/71a74e410df72962b207800bda2ba5881f0c610b) 12552 test fixes - [`a480a4c`](https://github.com/netbox-community/netbox/commit/a480a4c23b247a55afb8a52311629855f372e64b) 12552 add back add_related_count - [`3bd3668`](https://github.com/netbox-community/netbox/commit/3bd366893c76aa280f4d88e60797baa6a50e3731) 12552 add back add_related_count - [`9ee6511`](https://github.com/netbox-community/netbox/commit/9ee6511194a2d48215c9c409ea66eb363e1f7b17) 12552 remove extra param - [`ff9a9d9`](https://github.com/netbox-community/netbox/commit/ff9a9d914fea5c2c648e55cdafb3a1dc78dbf566) 12552 cleanup mptt - [`a8c9d87`](https://github.com/netbox-community/netbox/commit/a8c9d87a5cbab75ef815945183501abf67199524) 12552 test fixes - [`786394f`](https://github.com/netbox-community/netbox/commit/786394fbe8c733b7dce91b40cf5943d81e3ef71f) add back mptt to requirements ### 📊 Changes **37 files changed** (+463 additions, -134 deletions) <details> <summary>View changed files</summary> 📝 `base_requirements.txt` (+4 -0) 📝 `netbox/dcim/api/nested_serializers.py` (+5 -5) 📝 `netbox/dcim/api/serializers.py` (+2 -2) ➕ `netbox/dcim/migrations/0174_mptt_to_cte.py` (+164 -0) 📝 `netbox/dcim/models/device_component_templates.py` (+4 -12) 📝 `netbox/dcim/models/device_components.py` (+5 -13) 📝 `netbox/dcim/models/devices.py` (+1 -1) 📝 `netbox/dcim/models/sites.py` (+4 -3) 📝 `netbox/dcim/signals.py` (+2 -2) 📝 `netbox/dcim/tables/devices.py` (+1 -1) 📝 `netbox/dcim/tables/sites.py` (+3 -3) 📝 `netbox/dcim/views.py` (+3 -3) 📝 `netbox/extras/querysets.py` (+6 -4) 📝 `netbox/ipam/querysets.py` (+5 -5) 📝 `netbox/netbox/api/serializers/__init__.py` (+2 -2) 📝 `netbox/netbox/models/__init__.py` (+5 -24) 📝 `netbox/netbox/tables/columns.py` (+4 -4) 📝 `netbox/templates/dcim/site.html` (+1 -1) 📝 `netbox/tenancy/api/nested_serializers.py` (+2 -2) ➕ `netbox/tenancy/migrations/0011_mptt_to_cte.py` (+67 -0) _...and 17 more files_ </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 opening a pull request. This helps avoid waste time and effort on a proposed change that we might not be able to accept. IF YOUR PULL REQUEST DOES NOT REFERENCE AN ISSUE WHICH HAS BEEN ASSIGNED TO YOU, IT WILL BE CLOSED AUTOMATICALLY. Please specify your assigned issue number on the line below. --> ### Fixes: #12552 <!-- Please include a summary of the proposed changes below. --> --- <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:22:09 +01:00
adam closed this issue 2025-12-29 23:22:09 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#14034