[PR #3069] [MERGED] [management] add activity events for networks #21386

Open
opened 2026-08-05 05:08:26 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/3069
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-events


📝 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
  • 923e8fe add mock and fix test
  • 69f058c remove unused variable

📊 Changes

11 files changed (+373 additions, -59 deletions)

View changed files

📝 management/cmd/management.go (+2 -2)
📝 management/server/activity/codes.go (+30 -0)
📝 management/server/groups/manager.go (+45 -12)
📝 management/server/networks/manager.go (+55 -9)
📝 management/server/networks/manager_test.go (+21 -10)
📝 management/server/networks/resources/manager.go (+84 -19)
📝 management/server/networks/resources/types/resource.go (+4 -0)
📝 management/server/networks/routers/manager.go (+116 -7)
📝 management/server/networks/routers/types/router.go (+4 -0)
📝 management/server/networks/types/network.go (+4 -0)
📝 management/server/status/error.go (+8 -0)

📄 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/3069 **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-events` --- ### 📝 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 - [`923e8fe`](https://github.com/netbirdio/netbird/commit/923e8fe4e90b41438cb3c55971b538949c59ad88) add mock and fix test - [`69f058c`](https://github.com/netbirdio/netbird/commit/69f058cbe18da4dac57f1395b8df22704ef6739f) remove unused variable ### 📊 Changes **11 files changed** (+373 additions, -59 deletions) <details> <summary>View changed files</summary> 📝 `management/cmd/management.go` (+2 -2) 📝 `management/server/activity/codes.go` (+30 -0) 📝 `management/server/groups/manager.go` (+45 -12) 📝 `management/server/networks/manager.go` (+55 -9) 📝 `management/server/networks/manager_test.go` (+21 -10) 📝 `management/server/networks/resources/manager.go` (+84 -19) 📝 `management/server/networks/resources/types/resource.go` (+4 -0) 📝 `management/server/networks/routers/manager.go` (+116 -7) 📝 `management/server/networks/routers/types/router.go` (+4 -0) 📝 `management/server/networks/types/network.go` (+4 -0) 📝 `management/server/status/error.go` (+8 -0) </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:08:26 -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#21386