Skip to content

PR_26174_ALFA_017-game-hub-guest-save-and-crew-cleanup#111

Closed
ToolboxAid wants to merge 1 commit into
pr/26174-ALFA-016-game-hub-row-edit-add-selected-statefrom
pr/26174-ALFA-017-game-hub-guest-save-and-crew-cleanup
Closed

PR_26174_ALFA_017-game-hub-guest-save-and-crew-cleanup#111
ToolboxAid wants to merge 1 commit into
pr/26174-ALFA-016-game-hub-row-edit-add-selected-statefrom
pr/26174-ALFA-017-game-hub-guest-save-and-crew-cleanup

Conversation

@ToolboxAid

Copy link
Copy Markdown
Owner

Summary

  • redirects guest Add/Edit Save actions to account/sign-in.html
  • moves the current role controls under a Game Crew accordion
  • removes Owner from displayed Game Hub parent table fields so columns are Game, Purpose, Status, Actions
  • keeps Source Idea and Readiness Output as child tables under each game row
  • adds targeted Playwright coverage and required ALFA_017 reports/ZIP

Validation

  • git diff --check -- toolbox/game-hub/index.html toolbox/game-hub/game-hub.js tests/playwright/tools/GameHubMockRepository.spec.mjs
  • node --check toolbox/game-hub/game-hub.js
  • npx playwright test tests/playwright/tools/GameHubMockRepository.spec.mjs -g "Game Hub" (11 passed)

Artifact

  • tmp/PR_26174_ALFA_017-game-hub-guest-save-and-crew-cleanup_delta.zip

Notes

A broader unscoped run of npx playwright test tests/playwright/tools/GameHubMockRepository.spec.mjs was attempted first. It reported 12 passed and 2 failed; the ALFA_017 guest redirect assertion was fixed, and the remaining failure was outside this PR surface: Toolbox member-role filters focus tools without exposing admin-only controls received existing 500 /api/game-journey/completion-metrics responses.

@ToolboxAid

Copy link
Copy Markdown
Owner Author

OWNER-approved closure: this historical Alfa PR is superseded by merged consolidated PR #149. No branch deletion requested.

@ToolboxAid ToolboxAid closed this Jun 24, 2026
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.

2 participants