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

refactor(console): refactor SIE settings form #7154

Merged

Conversation

simeng-li
Copy link
Contributor

Summary

This PR includes the following updates:

Core

Removed the restriction rule in the sign-in experience settings:

  • Previously, if the password was set as mandatory in the sign-up settings, all sign-in methods were required to enable password verification. For better flexibility, this restriction has now been removed.

Console

Refactor the console sign-in and sign-up settings page to align with the recent sign-in experience validation rule updates.

  1. Password is no longer mandatory when using username as a sign-up identifier.
    The password option in the sign-up settings is now editable. Note: When username is selected as a sign-up identifier, the system will still auto-enable the password option by default. However, users can manually deselect it if they prefer.

  2. Sign-up identifiers are no longer required in sign-in methods.
    All sign-in methods are now optional. Newly assigned sign-up identifiers will still be auto-appended to the sign-in methods list if they are not already included. However, users can manually remove them if they choose not to support them for sign-in.

  3. Password is no longer mandatory for email and phone number sign-in methods, even when password verification is enabled for sign-up.
    The password option is now editable for email and phone number sign-in methods, giving users more flexibility to configure their preferred sign-in experience.

Testing

test locally

Checklist

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

@simeng-li simeng-li requested review from a team March 18, 2025 11:35
Copy link

github-actions bot commented Mar 18, 2025

COMPARE TO master

Total Size Diff ⚠️ 📈 +20.24 KB

Diff by File
Name Diff
packages/console/src/pages/SignInExperience/PageContent/SignUpAndSignIn/SignInForm/SignInMethodEditBox/index.tsx 📈 +465 Bytes
packages/console/src/pages/SignInExperience/PageContent/SignUpAndSignIn/SignUpForm/SignUpForm.tsx 📈 +6.59 KB
packages/console/src/pages/SignInExperience/PageContent/SignUpAndSignIn/SignUpForm/SignUpIdentifiersEditBox/SignUpIdentifierItem.module.scss 📈 +612 Bytes
packages/console/src/pages/SignInExperience/PageContent/SignUpAndSignIn/SignUpForm/SignUpIdentifiersEditBox/SignUpIdentifierItem.tsx 📈 +1.61 KB
packages/console/src/pages/SignInExperience/PageContent/SignUpAndSignIn/SignUpForm/SignUpIdentifiersEditBox/index.module.scss 📈 +56 Bytes
packages/console/src/pages/SignInExperience/PageContent/SignUpAndSignIn/SignUpForm/SignUpIdentifiersEditBox/index.tsx 📈 +4.45 KB
packages/console/src/pages/SignInExperience/PageContent/SignUpAndSignIn/SignUpForm/index.tsx 📈 +699 Bytes
packages/console/src/pages/SignInExperience/PageContent/SignUpAndSignIn/components/IdentifiersAddButton/index.module.scss 📈 +102 Bytes
packages/console/src/pages/SignInExperience/PageContent/SignUpAndSignIn/components/IdentifiersAddButton/index.tsx 📈 +2.08 KB
packages/console/src/pages/SignInExperience/PageContent/SignUpAndSignIn/index.tsx 📈 +151 Bytes
packages/console/src/pages/SignInExperience/PageContent/SignUpAndSignIn/utils.ts 📈 +230 Bytes
packages/console/src/pages/SignInExperience/PageContent/constants.ts 📈 +108 Bytes
packages/console/src/pages/SignInExperience/PageContent/utils/parser.ts 📈 +2.67 KB
packages/console/src/pages/SignInExperience/types.ts 📈 +753 Bytes
packages/core/src/libraries/sign-in-experience/sign-in.test.ts 📈 +670 Bytes
packages/core/src/libraries/sign-in-experience/sign-in.ts 📈 +213 Bytes
packages/integration-tests/src/tests/console/sign-in-experience/sign-up-and-sign-in/happy-path.test.ts 📈 +303 Bytes
packages/integration-tests/src/tests/console/sign-in-experience/sign-up-and-sign-in/sad-path.test.ts 📈 +93 Bytes
packages/integration-tests/src/utils.ts 📈 +163 Bytes
packages/phrases/src/locales/en/translation/admin-console/sign-in-exp/sign-up-and-sign-in.ts 📈 +3 Bytes
packages/schemas/src/foundations/jsonb-types/sign-in-experience.ts 📈 +50 Bytes

@simeng-li simeng-li force-pushed the simeng-log-10898-console-refactor-sign-up-and-sign-in-form branch from 697d3c4 to 6df802e Compare March 21, 2025 08:50
@simeng-li simeng-li requested a review from darcyYe as a code owner March 21, 2025 08:50
@simeng-li simeng-li force-pushed the simeng-log-10932-experience-update-the-experience-sign-up-form-render-logic branch 2 times, most recently from e26d414 to 89a21e2 Compare March 24, 2025 02:22
@simeng-li simeng-li force-pushed the simeng-log-10898-console-refactor-sign-up-and-sign-in-form branch from 6df802e to ba151b0 Compare March 24, 2025 02:24
@simeng-li simeng-li force-pushed the simeng-log-10932-experience-update-the-experience-sign-up-form-render-logic branch from 89a21e2 to 8a38d05 Compare March 24, 2025 03:49
@simeng-li simeng-li force-pushed the simeng-log-10898-console-refactor-sign-up-and-sign-in-form branch from ba151b0 to acdab07 Compare March 24, 2025 03:49
Base automatically changed from simeng-log-10932-experience-update-the-experience-sign-up-form-render-logic to master March 24, 2025 06:01
@github-actions github-actions bot added size/l and removed size/m labels Mar 24, 2025
@simeng-li simeng-li force-pushed the simeng-log-10898-console-refactor-sign-up-and-sign-in-form branch from acdab07 to 3d7aab7 Compare March 24, 2025 06:03
@github-actions github-actions bot added size/m and removed size/l labels Mar 24, 2025
update SIE page to support multi sign-up identifiers
@github-actions github-actions bot removed the size/m label Mar 24, 2025
disable test cases for devFeature
disable some console tests for devFeature
fix emailOrPhone sign-in method sync
@simeng-li simeng-li merged commit cdc1acb into master Mar 25, 2025
36 checks passed
@simeng-li simeng-li deleted the simeng-log-10898-console-refactor-sign-up-and-sign-in-form branch March 25, 2025 01:53
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.

3 participants