feat: add prebuilt templates to support form #316
Claude / Claude Code Review
completed
May 5, 2026 in 8m 34s
Code review found 1 potential issue
Found 5 candidates, confirmed 1. See review comments for details.
Details
| Severity | Count |
|---|---|
| 🔴 Important | 0 |
| 🟡 Nit | 1 |
| 🟣 Pre-existing | 0 |
| Severity | File:Line | Issue |
|---|---|---|
| 🟡 Nit | src/features/dashboard/navbar/report-issue-dialog.tsx:149-163 |
Switching back to "Something else" surfaces validation error before user input |
Annotations
Check warning on line 163 in src/features/dashboard/navbar/report-issue-dialog.tsx
claude / Claude Code Review
Switching back to "Something else" surfaces validation error before user input
When a user picks a non-default template (so the prefill is applied) and then switches back to **Something else**, `handleTemplateChange` runs `form.setValue('description', '', { shouldValidate: true })`. The empty string immediately fails the `description.min(1)` rule and `FormMessage` renders *Please describe how we can help* under the textarea before the user has typed anything. Consider passing `shouldValidate: false` (or skipping the `setValue` call) when transitioning to the empty default
Loading