con2 / con2/kompassi

Perk override UI allows negative values (Ropecon: -2 meal tickets)

Open
#1,028 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
29
Forks
33
Avg merge
1d 22h
Merged PRs (30d)
19

Description

Reported by Ropecon 2026 in their post-con feedback: several program hosts ended up with exactly -2 meal tickets in their perks. Most likely user error, but the UI permits it.

The perk override number input (`src/components/involvement/PerksForm.tsx`, ``) has no `min` attribute and no clamping. A plausible mechanism for accidental changes: some browsers adjust a focused number input's value on mouse wheel scroll.

Fix:
* `min={0}` on the number input and clamp on change.
* Validate non-negative perk counts in the backend override mutation as well.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.