Skip to content

feat: manage games with module usage and a delete warning - #170

Merged
sean-dickinson merged 2 commits into
contentable-v2-student-homefrom
contentable-v2-games-admin
Nov 1, 2026
Merged

sean-dickinson merged 2 commits into
contentable-v2-student-homefrom
contentable-v2-games-admin

Conversation

@sean-dickinson

@sean-dickinson sean-dickinson commented Oct 31, 2026 •

Copy link
Copy Markdown
Collaborator

Description

Layer 7 of 7 in the contentable stack.

The games admin gets a nav link and pages that match the other admin pages.

  • The show page lists the modules that use the game.
  • A counter cache tracks student attempts.
  • The delete confirmation says how many modules and student attempts the delete affects.

Open question for the client: when students have attempts, do we block the delete of a game (dependent: :restrict_with_error)? Now the attempts are deleted with the game, and a model test documents this.

Type of change

  • New feature

How Has This Been Tested?

bin/ci passes on this branch.

🤖 Generated with Claude Code

@sean-dickinson
sean-dickinson added this pull request to stack #171 October 31, 2026 20:43
@sean-dickinson
sean-dickinson force-pushed the contentable-v2-games-admin branch from 0b21765 to 4e35e6d Compare October 31, 2026 23:58
@sean-dickinson
sean-dickinson force-pushed the contentable-v2-games-admin branch from 4e35e6d to c88c8f5 Compare November 1, 2026 19:50
@sean-dickinson
sean-dickinson force-pushed the contentable-v2-games-admin branch 2 times, most recently from 55c96a2 to dea4932 Compare November 1, 2026 20:27
@sean-dickinson
sean-dickinson force-pushed the contentable-v2-games-admin branch from dea4932 to c99d976 Compare November 1, 2026 20:47
Adds a games admin section with module usage, an attempts counter cache, and a delete warning.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sean-dickinson
sean-dickinson force-pushed the contentable-v2-games-admin branch from c99d976 to 194dcf9 Compare November 1, 2026 21:10
@sean-dickinson
sean-dickinson marked this pull request as ready for review November 1, 2026 21:28
@sean-dickinson
sean-dickinson merged commit f4d45f7 into main Nov 1, 2026
4 of 5 checks passed
@sean-dickinson
sean-dickinson deleted the contentable-v2-games-admin branch November 1, 2026 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant