chore: AI feedback

Author: brobro10000

src/components/FormFields/RegisterAccountFields.tsx

@@ -3,7 +3,7 @@ import { Stack } from '@openedx/paragon';
 import { Lock, Visibility, VisibilityOff } from '@openedx/paragon/icons';
 import { useState } from 'react';
 
-import { useCountryOptions, useRecaptchaToken } from '@/components/app/data';
+import { useCountryOptions } from '@/components/app/data';
 import { FieldContainer } from '@/components/FieldContainer';
 import Field from '@/components/FormFields/Field';
 
@@ -230,24 +230,18 @@ export const RegisterAccountCountry = ({
   );
 };
 
-const RegisterAccountFields = ({ form }: RegisterAccountFieldsProps) => {
-  const { isLoading } = useRecaptchaToken('submit');
-  if (isLoading) {
-    return null;
-  }
-  return (
-    <FieldContainer>
-      <RegisterAccountFieldsHeader />
-      <Stack gap={3}>
-        <RegisterAccountAdminEmail form={form} />
-        <RegisterAccountFullName form={form} />
-        <RegisterAccountUsername form={form} />
-        <RegisterAccountPassword form={form} />
-        <RegisterAccountConfirmPassword form={form} />
-        <RegisterAccountCountry form={form} />
-      </Stack>
-    </FieldContainer>
-  );
-};
+const RegisterAccountFields = ({ form }: RegisterAccountFieldsProps) => (
+  <FieldContainer>
+    <RegisterAccountFieldsHeader />
+    <Stack gap={3}>
+      <RegisterAccountAdminEmail form={form} />
+      <RegisterAccountFullName form={form} />
+      <RegisterAccountUsername form={form} />
+      <RegisterAccountPassword form={form} />
+      <RegisterAccountConfirmPassword form={form} />
+      <RegisterAccountCountry form={form} />
+    </Stack>
+  </FieldContainer>
+);
 
 export default RegisterAccountFields;

src/components/app/data/hooks/useRecaptchaToken.tsx

@@ -1,5 +1,5 @@
 import { getConfig } from '@edx/frontend-platform';
-import { logError } from '@edx/frontend-platform/logging';
+import { logError, logInfo } from '@edx/frontend-platform/logging';
 import { useCallback } from 'react';
 import { useGoogleReCaptcha } from 'react-google-recaptcha-v3';
 
@@ -14,15 +14,14 @@ const useRecaptchaToken = (actionName = 'submit') => {
     if (isReady) {
       const token = await executeRecaptcha(actionName);
       if (!token) {
-        throw new Error("Oopsie! reCAPTCHA didn't return a token.");
+        throw new Error(`Failed to obtain reCAPTCHA verification token for action: ${actionName}. Please try again or contact support if the issue persists.`);
       }
       return token;
     }
 
     // Fallback: no reCAPTCHA or not ready
     if (RECAPTCHA_SITE_KEY_WEB) {
-      // eslint-disable-next-line no-console
-      console.warn(`reCAPTCHA not ready for action: ${actionName}. Proceeding without token.`);
+      logInfo(`reCAPTCHA not ready for action: ${actionName}. Proceeding without token.`);
     }
     return null;
   }, [isReady, RECAPTCHA_SITE_KEY_WEB, executeRecaptcha, actionName]);

src/components/app/data/services/registration.ts

@@ -70,14 +70,9 @@ declare global {
     country: string;
   }
 
-  interface RegistrationCreateRecaptchaRequestSchema extends BaseRegistrationCreateRequestSchema {
-    recaptchaToken: string;
-  }
-
-  interface RegistrationCreateRequestSchema extends
-    BaseRegistrationCreateRequestSchema,
-    RegistrationCreateRecaptchaRequestSchema {
+  interface RegistrationCreateRequestSchema extends BaseRegistrationCreateRequestSchema {
     honorCode: boolean;
+    recaptchaToken?: string;
   }
 
   interface RegistrationCreateSuccessResponseSchema {

src/components/plan-details-pages/PlanDetailsPage.tsx

@@ -175,7 +175,7 @@ const PlanDetailsPage = () => {
     [SubmitCallbacks.PlanDetailsRegister]: async (data: PlanDetailsRegisterPageData) => {
       const recaptchaToken: string | null = await getToken();
 
-      let registerMutationPayload: Partial<RegistrationCreateRequestSchema> = {
+      const registerMutationPayload: Partial<RegistrationCreateRequestSchema> = {
         name: data.fullName,
         email: data.adminEmail,
         username: data.username,
@@ -184,10 +184,7 @@ const PlanDetailsPage = () => {
       };
 
       if (recaptchaToken) {
-        registerMutationPayload = {
-          ...registerMutationPayload,
-          recaptchaToken,
-        } as RegistrationCreateRecaptchaRequestSchema;
+        registerMutationPayload.recaptchaToken = recaptchaToken;
       }
       registerMutation.mutate(registerMutationPayload);
     },