Skip to content

Allow assigning competitors directly to round attempt for 333fm and 333mbf#80

Open
Copilot wants to merge 10 commits intomainfrom
copilot/allow-assign-people-to-round
Open

Allow assigning competitors directly to round attempt for 333fm and 333mbf#80
Copilot wants to merge 10 commits intomainfrom
copilot/allow-assign-people-to-round

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 7, 2026

  • Inspect latest CI workflow runs and identify exact failing test job/logs
  • Reproduce failing tests locally and isolate root cause
  • Update round selector UI so attempts are nested under their parent round
  • Ensure round-level assignment action is visible and usable in configure assignments flow
  • Update/add focused tests for nested round-attempt behavior
  • Re-run targeted tests and final validations
  • Reply to PR comment with commit hash

Copilot AI changed the title [WIP] Add option to assign people directly to round for mbld and 333fm Allow assigning competitors directly to round attempt for 333fm and 333mbf May 7, 2026
Copilot AI requested a review from coder13 May 7, 2026 15:19
Copilot finished work on behalf of coder13 May 7, 2026 15:19
Copilot finished work on behalf of coder13 May 7, 2026 16:46
@coder13
Copy link
Copy Markdown
Owner

coder13 commented May 7, 2026

@copilot Please fix lint / type check issues

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 7, 2026

@copilot Please fix lint / type check issues

Fixed in 984899f. I updated .github/workflows/lint.yml and .github/workflows/type-check.yml to use node-version-file: '.nvmrc' (instead of Node 18), which resolves the lint/type-check CI install/typecheck failures caused by Node engine mismatch. No UI changes in this update.

Copilot finished work on behalf of coder13 May 7, 2026 17:24
@coder13
Copy link
Copy Markdown
Owner

coder13 commented May 7, 2026

@copilot fix tests

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 7, 2026

@copilot fix tests

Fixed in 2ce569e (with follow-up cleanup in f92e114). I updated src/store/actions.test.ts to match the current WCIF patch payload (including formatVersion), and test/lint/type-check now pass in CI. I also applied the requested UI adjustment so attempt rows are nested under the round and round-level assignment actions are clearer.

Copilot finished work on behalf of coder13 May 7, 2026 23:42
@coder13 coder13 marked this pull request as ready for review May 8, 2026 01:07
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.

Allow assigning people to the round for mbld and 333fm

2 participants