Skip to content

Agent modal inline validation was replaced with the generic hook#3627

Merged
Br0wnHammer merged 1 commit intodevelopfrom
feature/agent-modal-replace-inline-validation-with-generic-hook
Mar 30, 2026
Merged

Agent modal inline validation was replaced with the generic hook#3627
Br0wnHammer merged 1 commit intodevelopfrom
feature/agent-modal-replace-inline-validation-with-generic-hook

Conversation

@Innagl
Copy link
Copy Markdown
Contributor

@Innagl Innagl commented Mar 27, 2026

Describe your changes

A reusable generic useFormValidation hook has been integrated into the "Add agent" modal, replacing the previous custom validation logic.

Write your issue number after "Fixes "

Fixes #3619

Please ensure all items are checked off before requesting a review:

  • I deployed the code locally.
  • I have performed a self-review of my code.
  • I have included the issue # in the PR.
  • I have labelled the PR correctly.
  • The issue I am working on is assigned to me.
  • I have avoided using hardcoded values to ensure scalability and maintain consistency across the application.
  • I have ensured that font sizes, color choices, and other UI elements are referenced from the theme.
  • My pull request is focused and addresses a single, specific feature.
  • If there are UI changes, I have attached a screenshot or video to this PR.
Screenshot 2026-03-27 at 6 28 45 PM

@Innagl Innagl added this to the 2.3 milestone Mar 27, 2026
@Innagl Innagl self-assigned this Mar 27, 2026
@Innagl Innagl added enhancement New feature or request frontend Frontend related tasks/issues labels Mar 27, 2026
Copy link
Copy Markdown
Member

@Br0wnHammer Br0wnHammer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

@Br0wnHammer Br0wnHammer merged commit 135d5d1 into develop Mar 30, 2026
3 of 4 checks passed
@Br0wnHammer Br0wnHammer deleted the feature/agent-modal-replace-inline-validation-with-generic-hook branch March 30, 2026 04:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request frontend Frontend related tasks/issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add agent modal: replace local inline validateForm with shared hook

2 participants