Skip to content

PM-4835: remove inline group creation from challenge groups#1718

Merged
jmgasper merged 1 commit intodevfrom
PM-4835
Apr 10, 2026
Merged

PM-4835: remove inline group creation from challenge groups#1718
jmgasper merged 1 commit intodevfrom
PM-4835

Conversation

@jmgasper
Copy link
Copy Markdown
Collaborator

@jmgasper jmgasper commented Apr 10, 2026

What was broken
Users could type a new group name into the Create Challenge groups field and select the generated "Create ..." option, which led to a failing group-create request and no useful UI response.

Root cause (if identifiable)
The challenge groups field reused a shared async-creatable select, so it exposed inline group creation even though the challenge flow already has a separate modal-based group creation path.

What was changed
Added an explicit isCreatable switch to the shared work-app groups select and disabled inline creation for the challenge editor groups field, leaving the existing modal-based create-group flow in place.

Any added/updated tests
Updated the challenge groups field test to assert that the challenge editor passes isCreatable={false} while preserving the existing modal create-group behavior.


Open with Devin

What was broken
Users could type a new group name into the Create Challenge groups field and select the generated "Create ..." option, which led to a failing group-create request and no useful UI response.

Root cause (if identifiable)
The challenge groups field reused a shared async-creatable select, so it exposed inline group creation even though the challenge flow already has a separate modal-based group creation path.

What was changed
Added an explicit `isCreatable` switch to the shared work-app groups select and disabled inline creation for the challenge editor groups field, leaving the existing modal-based create-group flow in place.

Any added/updated tests
Updated the challenge groups field test to assert that the challenge editor passes `isCreatable={false}` while preserving the existing modal create-group behavior.
@jmgasper jmgasper requested a review from kkartunov as a code owner April 10, 2026 07:45
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@jmgasper jmgasper merged commit e63155f into dev Apr 10, 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