Quick wins: expansion arrows, multi-use invites, 404 page - #366
Merged
Conversation
The expandable request rows had no visual indicator they could be clicked. Add a CSS chevron (▸) that rotates when expanded.
Single-use invites were the only option. Server hosts often want a shareable link that multiple players can use. Add max_uses column (NULL = unlimited, integer = cap) and use_count tracking. Existing codes default to single-use for backward compat. Also registers migration 019 (queue unique constraints) which was present on disk but missing from the migrations array. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Unrecognized paths under /quma/ fell through to the SPT proxy, returning confusing proxy errors. Add a default_service on the /quma scope that renders the existing styled error template. Non-/quma paths still proxy to SPT as expected. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
max_uses/use_countcolumns with backward-compatible migration. UI shows uses dropdown on create form and X/Y counter in the table/quma/*routes now render the existing styled error template instead of falling through to the SPT proxy (non-/qumapaths still proxy to SPT correctly)019_queue_unique_constraints.sqlwhich was present on disk but missing from the migrations arrayTest plan
cargo test)cargo clippy -- -D warnings)/quma/nonexistentand confirm styled 404 page/launcher/pingand confirm SPT proxy still worksImplemented with the help of Claude Code