Skip to content

fix(clerk-js): Disable emailAdress field in SignUp if fields.emailAddress.disabled is true#2990

Merged
nikosdouvlis merged 1 commit intomainfrom
nikospap/core-1649-email-address-should-not-be-editable-if-coming-from-ticket
Mar 13, 2024
Merged

fix(clerk-js): Disable emailAdress field in SignUp if fields.emailAddress.disabled is true#2990
nikosdouvlis merged 1 commit intomainfrom
nikospap/core-1649-email-address-should-not-be-editable-if-coming-from-ticket

Conversation

@nikospapcom
Copy link
Copy Markdown
Contributor

@nikospapcom nikospapcom commented Mar 13, 2024

Description

In this pr we are disable the emailAdresss field in SignUp when coming from ticket flow and fields.emailAddress.disabled is true

Checklist

  • npm test runs as expected.
  • npm run 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:

BEFORE

Screenshot 2024-03-13 at 2 04 07 PM

Screenshot 2024-03-13 at 2 03 31 PM

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Mar 13, 2024

🦋 Changeset detected

Latest commit: a1cf685

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

Comment thread packages/clerk-js/src/ui/components/SignUp/SignUpForm.tsx
@nikospapcom nikospapcom self-assigned this Mar 13, 2024
@nikosdouvlis nikosdouvlis merged commit 7d3aa44 into main Mar 13, 2024
@nikosdouvlis nikosdouvlis deleted the nikospap/core-1649-email-address-should-not-be-editable-if-coming-from-ticket branch March 13, 2024 12:24
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