Skip to content

PM-4646: Require work type for design challenge creation#1575

Merged
jmgasper merged 3 commits intodevfrom
PM-4646
Apr 1, 2026
Merged

PM-4646: Require work type for design challenge creation#1575
jmgasper merged 3 commits intodevfrom
PM-4646

Conversation

@jmgasper
Copy link
Copy Markdown
Collaborator

@jmgasper jmgasper commented Apr 1, 2026

What was broken\nDesign challenge creation allowed the New action to succeed without a selected work type.\n\nRoot cause\nThe createNewChallenge flow validated only the basic fields it triggered and skipped workType even when the Design + Challenge selection made it mandatory.\n\nWhat was changed\nAdded a create-path validation guard for workType that uses the same Design + Challenge visibility condition as the UI before calling createChallenge.\nDocumented the new helper functions used by the create flow.\n\nAny added/updated tests\nUpdated ChallengeEditorForm.spec.tsx with a regression test that blocks design challenge creation when work type is missing.


Open with Devin

What was broken\nDesign challenge creation allowed the New action to succeed without a selected work type.\n\nRoot cause\nThe createNewChallenge flow validated only the basic fields it triggered and skipped workType even when the Design + Challenge selection made it mandatory.\n\nWhat was changed\nAdded a create-path validation guard for workType that uses the same Design + Challenge visibility condition as the UI before calling createChallenge.\nDocumented the new helper functions used by the create flow.\n\nAny added/updated tests\nUpdated ChallengeEditorForm.spec.tsx with a regression test that blocks design challenge creation when work type is missing.
@jmgasper jmgasper requested a review from kkartunov as a code owner April 1, 2026 13:41
devin-ai-integration[bot]

This comment was marked as resolved.

@jmgasper jmgasper merged commit 2bb5e57 into dev Apr 1, 2026
9 checks passed
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