[PR #8269] [MERGED] Add Cisco StackWise-n choices #13340

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

📋 Pull Request Information

Original PR: https://github.com/netbox-community/netbox/pull/8269
Author: @bluikko
Created: 1/7/2022
Status: Merged
Merged: 1/8/2022
Merged by: @DanSheps

Base: developHead: cisco-stackwise-n


📝 Commits (1)

  • b63e296 Add Cisco StackWise-n choices

📊 Changes

1 file changed (+8 additions, -0 deletions)

View changed files

📝 netbox/dcim/choices.py (+8 -0)

📄 Description

Fixes: #8265

Add the 4 new Catalyst 9xxx StackWise types.

I struggled to decide whether these should be listed above or below the FlexStack choices. But then I noticed how some other choices are grouped, for example: t1 > e1 > t3 > e3 and decided to put them below the FlexStack.


🔄 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/8269 **Author:** [@bluikko](https://github.com/bluikko) **Created:** 1/7/2022 **Status:** ✅ Merged **Merged:** 1/8/2022 **Merged by:** [@DanSheps](https://github.com/DanSheps) **Base:** `develop` ← **Head:** `cisco-stackwise-n` --- ### 📝 Commits (1) - [`b63e296`](https://github.com/netbox-community/netbox/commit/b63e29610e05a1518fd07dab0a2bacede6228cd4) Add Cisco StackWise-n choices ### 📊 Changes **1 file changed** (+8 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `netbox/dcim/choices.py` (+8 -0) </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 opened for approval prior to filing a pull request. This helps avoid wasting time and effort on something that we might not be able to accept. Please indicate the relevant feature request or bug report below. IF YOUR PULL REQUEST DOES NOT REFERENCE AN ACCEPTED BUG REPORT OR FEATURE REQUEST, IT WILL BE MARKED AS INVALID AND CLOSED. --> ### Fixes: #8265 <!-- Please include a summary of the proposed changes below. --> Add the 4 new Catalyst 9xxx StackWise types. I struggled to decide whether these should be listed above or below the FlexStack choices. But then I noticed how some other choices are grouped, for example: t1 > e1 > t3 > e3 and decided to put them below the FlexStack. --- <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 22:26:51 +01:00
adam closed this issue 2025-12-29 22:26:51 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#13340