Optional quantity with setMinValue(1) and default 1 preserves today’s behaviour when omitted.
Inventory updates use AddQuantity / constructor initial quantity consistently; reply includes amount granted and new total.
New unit tests cover default quantity, explicit quantity, invalid/negative quantity, and existing admin/sync/safe-mode/card guards.

Nits (non-blocking): The PR title still says “Start of …” though the issue subtasks look complete; consider retitling before merge. The Unreleased changelog block bundles several unrelated entries beyond #359 — fine if this targets release/0.11.0, but worth keeping scoped if you prefer smaller PRs.

**Approved** — `/give card` quantity support matches #359. - Optional `quantity` with `setMinValue(1)` and default 1 preserves today’s behaviour when omitted. - Inventory updates use `AddQuantity` / constructor initial quantity consistently; reply includes amount granted and new total. - New unit tests cover default quantity, explicit quantity, invalid/negative quantity, and existing admin/sync/safe-mode/card guards. **Nits (non-blocking):** The PR title still says “Start of …” though the issue subtasks look complete; consider retitling before merge. The `Unreleased` changelog block bundles several unrelated entries beyond #359 — fine if this targets `release/0.11.0`, but worth keeping scoped if you prefer smaller PRs.

Vylpes merged commit ec12facdd7 into release/0.11.0

2026-10-20 12:15:38 +01:00

Vylpes deleted branch feature/359-give-card-quantity

2026-10-20 12:15:38 +01:00

Vylpes referenced this pull request from a commit

2026-10-20 12:15:38 +01:00

Merge pull request 'Give command multiple quantities' (#568) from feature/359-give-card-quantity into release/0.11.0

Smithy-bot referenced this pull request

2026-10-23 12:03:47 +01:00

/Give card command should let you add multiple at once #359