[PR #534] [MERGED] Add Group to new client #534

New issue

Closed

opened 2026-02-28 00:42:33 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 00:42:33 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Telmate/proxmox-api-go/pull/534
Author: @Tinyblargon
Created: 12/29/2025
Status: ✅ Merged
Merged: 12/29/2025
Merged by: @Tinyblargon

Base: master ← Head: #483-group

📝 Commits (4)

df50017 feat: Add group to new Client
ce48100 perf: store token name when possible
2390ad9 refactor: fix typo in test name
eb20c2d refactor: fix typo in test name

📊 Changes

31 files changed (+2608 additions, -882 deletions)

View changed files

📝 cli/command/delete/delete-group.go (+2 -5)
📝 cli/command/delete/delete.go (+1 -1)
📝 cli/command/get/get-group.go (+2 -5)
📝 cli/command/get/get.go (+3 -1)
📝 cli/command/list/list-groups.go (+14 -12)
📝 cli/command/member/group/add-users.go (+6 -8)
📝 cli/command/member/group/remove-users.go (+6 -8)
📝 cli/command/member/group/set-users.go (+8 -9)
📝 cli/command/set/set-group.go (+3 -4)
📝 internal/array/array.go (+79 -1)
➕ internal/array/array_test.go (+175 -0)
📝 internal/mockServer/requests.go (+17 -4)
📝 proxmox/client.go (+1 -0)
📝 proxmox/client__api.go (+4 -0)
📝 proxmox/client__new.go (+2 -1)
📝 proxmox/config__apiToken.go (+11 -10)
📝 proxmox/config__apiToken_test.go (+8 -7)
📝 proxmox/config__group.go (+434 -229)
📝 proxmox/config__group_test.go (+974 -422)
📝 proxmox/config__user.go (+49 -69)

...and 11 more files

📄 Description

Re implements the group handling functionality in the New client interface.

Related to #483

_{🔄 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/Telmate/proxmox-api-go/pull/534 **Author:** [@Tinyblargon](https://github.com/Tinyblargon) **Created:** 12/29/2025 **Status:** ✅ Merged **Merged:** 12/29/2025 **Merged by:** [@Tinyblargon](https://github.com/Tinyblargon) **Base:** `master` ← **Head:** `#483-group` --- ### 📝 Commits (4) - [`df50017`](https://github.com/Telmate/proxmox-api-go/commit/df50017c63e6339735d66b1114b81af10ee9e289) feat: Add group to new Client - [`ce48100`](https://github.com/Telmate/proxmox-api-go/commit/ce481009ae8de2d24bb514b909ebb99a8c960ed2) perf: store token name when possible - [`2390ad9`](https://github.com/Telmate/proxmox-api-go/commit/2390ad9e331582dfa97c3aab39f6727c9d72f231) refactor: fix typo in test name - [`eb20c2d`](https://github.com/Telmate/proxmox-api-go/commit/eb20c2d4970268bc983197e863eee9f62a5dd08b) refactor: fix typo in test name ### 📊 Changes **31 files changed** (+2608 additions, -882 deletions) <details> <summary>View changed files</summary> 📝 `cli/command/delete/delete-group.go` (+2 -5) 📝 `cli/command/delete/delete.go` (+1 -1) 📝 `cli/command/get/get-group.go` (+2 -5) 📝 `cli/command/get/get.go` (+3 -1) 📝 `cli/command/list/list-groups.go` (+14 -12) 📝 `cli/command/member/group/add-users.go` (+6 -8) 📝 `cli/command/member/group/remove-users.go` (+6 -8) 📝 `cli/command/member/group/set-users.go` (+8 -9) 📝 `cli/command/set/set-group.go` (+3 -4) 📝 `internal/array/array.go` (+79 -1) ➕ `internal/array/array_test.go` (+175 -0) 📝 `internal/mockServer/requests.go` (+17 -4) 📝 `proxmox/client.go` (+1 -0) 📝 `proxmox/client__api.go` (+4 -0) 📝 `proxmox/client__new.go` (+2 -1) 📝 `proxmox/config__apiToken.go` (+11 -10) 📝 `proxmox/config__apiToken_test.go` (+8 -7) 📝 `proxmox/config__group.go` (+434 -229) 📝 `proxmox/config__group_test.go` (+974 -422) 📝 `proxmox/config__user.go` (+49 -69) _...and 11 more files_ </details> ### 📄 Description Re implements the group handling functionality in the New client interface. Related to #483 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>