Skip to content

Conversation

@nikospapcom
Copy link
Contributor

Description

In this pr we're displaying the validation fields errors in <Waitlist /> component. Before this pr we didn't display them correctly

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

AFTER

Screenshot 2025-02-27 at 5 11 47 PM

@nikospapcom nikospapcom self-assigned this Feb 27, 2025
@changeset-bot
Copy link

changeset-bot bot commented Feb 27, 2025

🦋 Changeset detected

Latest commit: d2640ef

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@clerk/clerk-js Patch
@clerk/chrome-extension Patch
@clerk/clerk-expo Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Feb 27, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
clerk-js-sandbox ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 4, 2025 8:58am

@nikospapcom nikospapcom force-pushed the nikospap/user-1266-email-validation-in-the-waitlist-component-isnt-displaying branch from 522b20e to 0404164 Compare March 4, 2025 08:41
@nikospapcom nikospapcom marked this pull request as ready for review March 4, 2025 08:54
@nikospapcom nikospapcom force-pushed the nikospap/user-1266-email-validation-in-the-waitlist-component-isnt-displaying branch from 0404164 to d2640ef Compare March 4, 2025 08:57
@nikospapcom nikospapcom merged commit 99f8a1c into main Mar 4, 2025
28 of 29 checks passed
@nikospapcom nikospapcom deleted the nikospap/user-1266-email-validation-in-the-waitlist-component-isnt-displaying branch March 4, 2025 09:22
jacekradko pushed a commit that referenced this pull request Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants