[PR #3070] [MERGED] [management] Include related policies into networks #19178

Closed
opened 2026-08-05 05:05:41 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/3070
Author: @pascal-fischer
Created: 12/17/2024
Status: Merged
Merged: 12/18/2024
Merged by: @pascal-fischer

Base: feature/new-networks-conceptHead: feature/new-networks-concept-policy


📝 Commits (10+)

  • dccbfe5 properly add and remove resources from groups
  • 22cbbf8 extract groups update into separate method
  • 40ab041 fix tests
  • cf73911 fix mock and error handling
  • 7bbdc3c fix nullpointer on group change while name change
  • 44fba5d fix manager test
  • fc1b4d7 add activity events
  • 95a18b9 add activity events
  • 3a4d37f expose related policies of a network
  • 923e8fe add mock and fix test

📊 Changes

7 files changed (+96 additions, -31 deletions)

View changed files

📝 management/server/http/api/openapi.yml (+7 -0)
📝 management/server/http/api/types.gen.go (+6 -3)
📝 management/server/http/handler.go (+1 -1)
📝 management/server/http/handlers/networks/handler.go (+43 -9)
📝 management/server/networks/types/network.go (+2 -1)
📝 management/server/route_test.go (+8 -7)
📝 management/server/types/account.go (+29 -10)

📄 Description

Describe your changes

Checklist

  • Is it a bug fix
  • Is a typo/documentation fix
  • Is a feature enhancement
  • It is a refactor
  • Created tests that fail without the change (if possible)
  • Extended the README / documentation, if necessary

🔄 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/netbirdio/netbird/pull/3070 **Author:** [@pascal-fischer](https://github.com/pascal-fischer) **Created:** 12/17/2024 **Status:** ✅ Merged **Merged:** 12/18/2024 **Merged by:** [@pascal-fischer](https://github.com/pascal-fischer) **Base:** `feature/new-networks-concept` ← **Head:** `feature/new-networks-concept-policy` --- ### 📝 Commits (10+) - [`dccbfe5`](https://github.com/netbirdio/netbird/commit/dccbfe526cfba3ea70f19c604545cb74af516f9d) properly add and remove resources from groups - [`22cbbf8`](https://github.com/netbirdio/netbird/commit/22cbbf889a84bec8f71fc612865f7e17f4322dda) extract groups update into separate method - [`40ab041`](https://github.com/netbirdio/netbird/commit/40ab0411984eab00c92b52bcd1f064339e5a80f5) fix tests - [`cf73911`](https://github.com/netbirdio/netbird/commit/cf739116cb853ff08488965a6a7c3a34583c0aca) fix mock and error handling - [`7bbdc3c`](https://github.com/netbirdio/netbird/commit/7bbdc3c5ac968a450509a348d0be9ef30419d3c9) fix nullpointer on group change while name change - [`44fba5d`](https://github.com/netbirdio/netbird/commit/44fba5df2e6ac48221fc7dbfc3d8099eb7f6215a) fix manager test - [`fc1b4d7`](https://github.com/netbirdio/netbird/commit/fc1b4d7b7b17100a30421c9ff8f4e73eb5970588) add activity events - [`95a18b9`](https://github.com/netbirdio/netbird/commit/95a18b93cbe28072044f4e40fee8814fc736d91d) add activity events - [`3a4d37f`](https://github.com/netbirdio/netbird/commit/3a4d37f41b109a3d1608cfada82ff6f7ff606933) expose related policies of a network - [`923e8fe`](https://github.com/netbirdio/netbird/commit/923e8fe4e90b41438cb3c55971b538949c59ad88) add mock and fix test ### 📊 Changes **7 files changed** (+96 additions, -31 deletions) <details> <summary>View changed files</summary> 📝 `management/server/http/api/openapi.yml` (+7 -0) 📝 `management/server/http/api/types.gen.go` (+6 -3) 📝 `management/server/http/handler.go` (+1 -1) 📝 `management/server/http/handlers/networks/handler.go` (+43 -9) 📝 `management/server/networks/types/network.go` (+2 -1) 📝 `management/server/route_test.go` (+8 -7) 📝 `management/server/types/account.go` (+29 -10) </details> ### 📄 Description ## Describe your changes ## Issue ticket number and link ### Checklist - [ ] Is it a bug fix - [ ] Is a typo/documentation fix - [ ] Is a feature enhancement - [ ] It is a refactor - [ ] Created tests that fail without the change (if possible) - [ ] Extended the README / documentation, if necessary --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2026-08-05 05:05:41 -04:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: DYNR/netbird#19178