Raise solution description limit to 600 characters - #401
Conversation
Align the solution description limit with the problem description limit. The public proposal form and the backoffice form capped input at 400 characters in the browser, while the backoffice update rule allowed 410. All three now allow 600, matching the existing rules for public proposal store and backoffice create. Add feature tests for the 600/601 boundary on both the public proposal store and the backoffice update. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> (cherry picked from commit 6454a86)
|
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (4)
WalkthroughThe solution description limit increased to 600 characters in update validation and both solution forms. Feature tests cover the 600-character boundary and reject 601-character descriptions. ChangesSolution description limit
Estimated code review effort: 2 (Simple) | ~10 minutes Change: Feature ✨ Finishing Touches📝 Generate docstrings
🧪 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 |



Summary
Cherry-pick of the production hotfix that raises the solution description limit from 400 to 600 characters, so
masterandproductiondo not drift.resources/views/solution/partials/new-solution-form.blade.php:maxlength, character counter and label now use 600.resources/views/backoffice/management/solution/create-edit/partials/basic-details.blade.php:maxlengthnow 600.app/Http/Controllers/Solution/SolutionController.php: backofficeupdaterule changed frommax:410tomax:600.Three feature tests cover the 600/601 boundary on the public proposal store and the backoffice update.
Summary by CodeRabbit
New Features
Bug Fixes
Tests