Skip to content

Add quit competitors audit log and history view - #307

Closed
Victus0660 wants to merge 3 commits into
thewca:mainfrom
Victus0660:feature/quit-history
Closed

Victus0660 wants to merge 3 commits into
thewca:mainfrom
Victus0660:feature/quit-history

Conversation

@Victus0660

Copy link
Copy Markdown
Contributor

Description

This PR resolves a pain point during scoretaking double-checking:

  1. Double-checking quit competitors: Allows scoretakers to easily confirm competitors who were quit/removed from a round via a dedicated history list, rather than checking for absence.
  2. Audit & Accountability: Provides Delegates with visibility into who performed each quit action, when it occurred, and whether a replacement competitor was added.

Changes

  • Added round_removals database migration and RoundRemoval Ecto schema.
  • Updated remove_person_from_round and remove_no_shows_from_round to log audit entries with current_user.
  • Exposed removals field on :round GraphQL schema.
  • Added QuitHistoryDialog component and integrated "Quit history" button into AdminRoundToolbar.

@jonatanklosko

jonatanklosko commented Aug 3, 2026 •

Copy link
Copy Markdown
Member

Hey @Victus0660, thanks for the PR! However, there is ongoing work on a live results platform integrated directly into the main WCA website (Integrated Live Results), and I believe the plan is to fully transition to that system in the upcoming months. For this reason, I'd rather avoid adding new features of this scope to WCA Live at this point.

You may have a look at https://github.com/thewca/worldcubeassociation.org. It may be a suitable to discuss the feature there, or it may even already be in place.

@Victus0660

Copy link
Copy Markdown
Contributor Author

@jonatanklosko
Thanks for the explanation and feedback! I completely understand the reasoning with Integrated Live Results on the way. I'll check out the main repository and see how I can contribute there instead. Cheers!

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