fix(clerk-js): Fix issue for legal consent checkbox showing up in whe…

…n disabled (#4403)
clerk · Oct 24, 2024 · d82886c · d82886c
1 parent 83c0e7f
commit d82886c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/.changeset/unlucky-coins-juggle.md b/.changeset/unlucky-coins-juggle.md
@@ -0,0 +1,5 @@
+---
+"@clerk/clerk-js": patch
+---
+
+Fix for legal consent rendered on the <SignUp/> component when only social sign up is enabled
diff --git a/packages/clerk-js/src/ui/components/SignUp/SignUpStart.tsx b/packages/clerk-js/src/ui/components/SignUp/SignUpStart.tsx
@@ -103,14 +103,15 @@ function _SignUpStart(): JSX.Element {
   const hasTicket = !!formState.ticket.value;
   const hasEmail = !!formState.emailAddress.value;
   const isProgressiveSignUp = userSettings.signUp.progressive;
+  const isLegalConsentEnabled = userSettings.signUp.legal_consent_enabled;
 
   const fields = determineActiveFields({
     attributes,
     hasTicket,
     hasEmail,
     activeCommIdentifierType,
     isProgressiveSignUp,
-    legalConsentRequired: userSettings.signUp.legal_consent_enabled,
+    legalConsentRequired: isLegalConsentEnabled,
   });
 
   const handleTokenFlow = () => {
@@ -293,7 +294,7 @@ function _SignUpStart(): JSX.Element {
                 />
               )}
             </SocialButtonsReversibleContainerWithDivider>
-            {!shouldShowForm && (
+            {!shouldShowForm && isLegalConsentEnabled && (
               <Form.ControlRow elementId='__experimental_legalAccepted'>
                 <LegalCheckbox
                   {...formState.__experimental_legalAccepted.props}