Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(experience): handle secondary sign-up identifiers #7142

Draft
wants to merge 5 commits into
base: simeng-log-10890-experience-api-update-registration-event-to-handle-multiple
Choose a base branch
from

Conversation

simeng-li
Copy link
Contributor

Summary

Update the use-sie hook to merge signUp.identifiers and signUp.secondaryIdentifiers. The combined list can be used to render the user registration form.

Testing

test locally

Checklist

  • .changeset
  • unit tests
  • integration tests
  • necessary TSDoc comments

Copy link

github-actions bot commented Mar 17, 2025

COMPARE TO master

Total Size Diff 📉 -6.6 KB

Diff by File
Name Diff
.changeset/purple-waves-sit.md 📉 -886 Bytes
Dockerfile 📉 -212 Bytes
packages/console/src/assets/images/recaptcha.svg 📉 -1.49 KB
packages/console/src/assets/images/turnstile.svg 📉 -1.39 KB
packages/console/src/components/Drawer/index.tsx 📉 -177 Bytes
packages/console/src/components/Region/index.tsx 📉 -486 Bytes
packages/console/src/components/Topbar/TenantSelector/TenantDropdownItem/index.tsx 📉 -28 Bytes
packages/console/src/consts/env.ts 📉 -102 Bytes
packages/console/src/hooks/use-console-routes/routes/security.tsx 📉 -229 Bytes
packages/console/src/pages/CaptchaDetails/CaptchaContent/index.tsx 📉 -2.81 KB
packages/console/src/pages/CaptchaDetails/index.module.scss 📉 -154 Bytes
packages/console/src/pages/CaptchaDetails/index.tsx 📉 -4.67 KB
packages/console/src/pages/Security/CaptchaCard/index.module.scss 📉 -250 Bytes
packages/console/src/pages/Security/CaptchaCard/index.tsx 📉 -1.12 KB
packages/console/src/pages/Security/CaptchaLogo/index.module.scss 📉 -163 Bytes
packages/console/src/pages/Security/CaptchaLogo/index.tsx 📉 -384 Bytes
packages/console/src/pages/Security/CreateCaptchaForm/ProviderRadioGroup/ProviderRadio/index.module.scss 📉 -533 Bytes
packages/console/src/pages/Security/CreateCaptchaForm/ProviderRadioGroup/ProviderRadio/index.tsx 📉 -801 Bytes
packages/console/src/pages/Security/CreateCaptchaForm/ProviderRadioGroup/index.module.scss 📉 -119 Bytes
packages/console/src/pages/Security/CreateCaptchaForm/ProviderRadioGroup/index.tsx 📉 -923 Bytes
packages/console/src/pages/Security/CreateCaptchaForm/constants.ts 📉 -821 Bytes
packages/console/src/pages/Security/CreateCaptchaForm/index.module.scss 📉 -113 Bytes
packages/console/src/pages/Security/CreateCaptchaForm/index.tsx 📉 -1.29 KB
packages/console/src/pages/Security/CreateCaptchaForm/types.ts 📉 -294 Bytes
packages/console/src/pages/Security/index.tsx 📉 -733 Bytes
packages/console/src/pages/Security/use-data-fetch.ts 📉 -765 Bytes
packages/console/src/pages/TenantSettings/Subscription/ConsoleEmbeddedPricing/Skeleton/index.module.scss 📉 -463 Bytes
packages/console/src/pages/TenantSettings/Subscription/ConsoleEmbeddedPricing/Skeleton/index.tsx 📉 -706 Bytes
packages/console/src/pages/TenantSettings/Subscription/ConsoleEmbeddedPricing/index.module.scss 📉 -185 Bytes
packages/console/src/pages/TenantSettings/Subscription/ConsoleEmbeddedPricing/index.tsx 📉 -4.24 KB
packages/console/src/pages/TenantSettings/Subscription/SwitchPlanActionBar/index.module.scss 📉 -37 Bytes
packages/console/src/pages/TenantSettings/Subscription/SwitchPlanActionBar/index.tsx 📉 -10 Bytes
packages/console/src/pages/TenantSettings/Subscription/index.tsx 📉 -139 Bytes
packages/console/src/pages/TenantSettings/TenantBasicSettings/ProfileForm/TenantRegion/index.tsx 0 Bytes
packages/console/vite.config.ts 📉 -122 Bytes
packages/core/src/libraries/sign-in-experience/sign-in.test.ts 📉 -2.54 KB
packages/core/src/libraries/sign-in-experience/sign-in.ts 📉 -908 Bytes
packages/core/src/libraries/sign-in-experience/sign-up.test.ts 📉 -3.24 KB
packages/core/src/libraries/sign-in-experience/sign-up.ts 📉 -2.2 KB
packages/core/src/routes/experience/classes/libraries/sign-in-experience-validator.test.ts 📉 -3.73 KB
packages/core/src/routes/experience/classes/libraries/sign-in-experience-validator.ts 📉 -1.76 KB
packages/core/src/routes/sign-in-experience/index.ts 📉 -46 Bytes
packages/experience/src/components/IdentifierRegisterForm/use-register-with-username.ts 📉 -1.15 KB
packages/experience/src/constants/env.ts 📉 -149 Bytes
packages/experience/src/containers/VerificationCode/use-register-flow-code-verification.ts 📉 -283 Bytes
packages/experience/src/containers/VerificationCode/use-sign-in-flow-code-verification.ts 📉 -44 Bytes
packages/experience/src/hooks/use-sie.ts 📉 -4.14 KB
packages/experience/src/pages/RegisterPassword/index.tsx 📉 -38 Bytes
packages/experience/src/pages/VerificationCode/index.tsx 📉 -75 Bytes
packages/experience/vite.config.ts 📉 -464 Bytes
packages/integration-tests/src/tests/api/experience-api/register-interaction/username-password.test.ts 📉 -2.08 KB
packages/integration-tests/src/tests/api/experience-api/register-interaction/verification-code.test.ts 📉 -2.36 KB
packages/integration-tests/src/tests/api/sign-in-experience.test.ts 📉 -93 Bytes
packages/phrases/src/locales/en/errors/sign-in-experiences.ts 📉 -155 Bytes
packages/phrases/src/locales/en/translation/admin-console/security.ts 📉 -1.06 KB
packages/schemas/src/foundations/jsonb-types/sign-in-experience.ts 📉 -1.88 KB

@simeng-li simeng-li marked this pull request as draft March 17, 2025 03:45
refactor the verification code sign-in and sign-up error handling logic.
@simeng-li simeng-li force-pushed the simeng-log-10932-experience-update-the-experience-sign-up-form-render-logic branch from 7525149 to 7d4b428 Compare March 17, 2025 07:27
remove verification code method assertion
@github-actions github-actions bot added size/m and removed size/s labels Mar 17, 2025
add some comments
update username registraction handler, directly submit the interaction is password is not enabled for sign-up
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

1 participant