[PR #10218] [MERGED] Closes #10043: Add support for 'limit' query parameter to available VLANs API endpoint #13584

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

📋 Pull Request Information

Original PR: https://github.com/netbox-community/netbox/pull/10218
Author: @jeremystretch
Created: 8/30/2022
Status: Merged
Merged: 8/30/2022
Merged by: @jeremystretch

Base: developHead: 10043-available-vlans-limit


📝 Commits (1)

  • de17a65 Closes #10043: Add support for 'limit' query parameter to available VLANs API endpoint

📊 Changes

3 files changed (+36 additions, -15 deletions)

View changed files

📝 docs/release-notes/version-3.3.md (+1 -0)
📝 netbox/ipam/api/views.py (+18 -11)
📝 netbox/ipam/tests/test_api.py (+17 -4)

📄 Description

Fixes: #10043

  • Refactor the limit calculation logic from AvailableIPAddressesView
  • Implement the same functionality on AvailableVLANsView
  • Update test_list_available_vlans() API test method

🔄 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/10218 **Author:** [@jeremystretch](https://github.com/jeremystretch) **Created:** 8/30/2022 **Status:** ✅ Merged **Merged:** 8/30/2022 **Merged by:** [@jeremystretch](https://github.com/jeremystretch) **Base:** `develop` ← **Head:** `10043-available-vlans-limit` --- ### 📝 Commits (1) - [`de17a65`](https://github.com/netbox-community/netbox/commit/de17a651e6f976e8b7c16b49d4e78f6a6988b870) Closes #10043: Add support for 'limit' query parameter to available VLANs API endpoint ### 📊 Changes **3 files changed** (+36 additions, -15 deletions) <details> <summary>View changed files</summary> 📝 `docs/release-notes/version-3.3.md` (+1 -0) 📝 `netbox/ipam/api/views.py` (+18 -11) 📝 `netbox/ipam/tests/test_api.py` (+17 -4) </details> ### 📄 Description ### Fixes: #10043 - Refactor the limit calculation logic from AvailableIPAddressesView - Implement the same functionality on AvailableVLANsView - Update `test_list_available_vlans()` API test method --- <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:19:42 +01:00
adam closed this issue 2025-12-29 23:19:42 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#13584