Skip to content

[F3-7] Update URL validation schemas and enhance modal functionality#31

Open
elsiedp wants to merge 3 commits intodevfrom
fix/f3-7-field-verification-social-media
Open

[F3-7] Update URL validation schemas and enhance modal functionality#31
elsiedp wants to merge 3 commits intodevfrom
fix/f3-7-field-verification-social-media

Conversation

@elsiedp
Copy link
Copy Markdown

@elsiedp elsiedp commented Mar 31, 2026

👋 TL;DR

  • Updated the AdminAOsModal to simplify query invalidation after updating AOs.
  • Introduced comprehensive URL validation schemas for website, Facebook, Instagram, and Twitter in the validators package.
  • Added tests for the new URL schemas to ensure proper validation of various URL formats.

🔎 Details

(coming soon)

✅ How to Test

(coming soon)

🥜 GIF

lack-of-hustle

- Updated the `AdminAOsModal` to simplify query invalidation after updating AOs.
- Introduced comprehensive URL validation schemas for website, Facebook, Instagram, and Twitter in the validators package.
- Added tests for the new URL schemas to ensure proper validation of various URL formats.
@elsiedp elsiedp requested a review from dnishiyama as a code owner March 31, 2026 08:40
@makeplane
Copy link
Copy Markdown

makeplane bot commented Mar 31, 2026

Linked to Plane Work Item(s)

This comment was auto-generated by Plane

…book, and Instagram to include explicit TypeScript type definitions for the `field` parameter, enhancing type safety and clarity.
@elsiedp elsiedp force-pushed the fix/f3-7-field-verification-social-media branch from 0057f9d to f7878e7 Compare March 31, 2026 15:45
… improve clarity and validation logic, ensuring proper handling of optional URLs and enhancing user experience with more descriptive error messages.
@elsiedp elsiedp force-pushed the fix/f3-7-field-verification-social-media branch from f7878e7 to ab547ce Compare March 31, 2026 15:49
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.

1 participant