feat: limit discrete questions to 50 options max#4336
feat: limit discrete questions to 50 options max#4336SylvainChevalier wants to merge 3 commits intomainfrom
Conversation
- Add guideline to question-writing page about discrete question limits - Add frontend validation in numeric_question_input to cap at 50 options - Add backend validation in QuestionWriteSerializer to enforce the limit - Update error messages in all translation files (en, es, cs, pt, zh, zh-TW) - Add MaxDiscreteOptionCount constant (frontend) and MAX_DISCRETE_OPTION_COUNT (backend) Closes #3780 Co-authored-by: Sylvain <SylvainChevalier@users.noreply.github.com>
|
@claude the error message and guidelines should say "increase the step size or tighten the bounds" |
|
Claude finished @SylvainChevalier's task in 3m 25s —— View job Updated error messages and guidelines
Changed "use a continuous question instead" to "tighten the bounds" across all locations:
|
Co-authored-by: Sylvain <SylvainChevalier@users.noreply.github.com>
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
🚀 Preview EnvironmentYour preview environment is ready!
Details
ℹ️ Preview Environment InfoIsolation:
Limitations:
Cleanup:
|
|
@claude functionality looks good, now fix the issues from the unit tests. |
|
Claude finished @SylvainChevalier's task in 5m 25s —— View job Fixed Prettier formatting issues
The Frontend Checks CI job was failing due to Prettier formatting issues in two files:
Both files now pass |
Closes #3780
Generated with Claude Code