Raise solution description limit to 600 characters - #400
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>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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
A client asked why solutions are capped at 400 characters while problems allow 600. There is no technical reason: the
descriptioncolumn onsolution_translationsis atextcolumn, and the server already accepted 600 characters on create. The 400 limit lived only in the two forms, and the backoffice update rule allowed 410.This PR sets the solution description limit to 600 everywhere, matching problems:
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. ThestoreanduserProposalStorerules already used 600.Three feature tests cover the 600/601 boundary on the public proposal store and the backoffice update.
This branch is based on
productionand contains only this change. The same commit is cherry-picked tomasterin a separate PR.