Skip to content

Conversation

@tomrndom
Copy link

@tomrndom tomrndom commented Feb 2, 2026

ref: https://app.clickup.com/t/86b7rg7ku

Signed-off-by: Tomás Castillo [email protected]

Summary by CodeRabbit

  • Bug Fixes
    • Improved sponsor settings form validation by aligning with standard behavior patterns. Removed manual suppression of blur-event validation, enabling the form to validate more consistently when users interact with input fields and transition between form elements.

Signed-off-by: Tomás Castillo <[email protected]>
@tomrndom tomrndom requested a review from smarcet February 2, 2026 15:39
@coderabbitai
Copy link

coderabbitai bot commented Feb 2, 2026

📝 Walkthrough

Walkthrough

Removed a single line disabling blur validation from the SponsorSettingsForm Formik configuration. The component will now apply Formik's default validation behavior on blur events instead of explicitly suppressing it.

Changes

Cohort / File(s) Summary
Form Configuration Update
src/components/forms/sponsor-settings-form/index.js
Removed validateOnBlur: false property from Formik initialization, allowing default blur validation behavior.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • smarcet

Poem

🐰 Hop along, dear form so bright,
No more silencing the blur's soft light,
Formik shall validate as it should,
One line removed makes all feel good!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and accurately describes the main change: removing the validateOnBlur: false override to allow validation on blur, which directly matches the code modification.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/sponsor-settings-email-blur-validation

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

2 participants