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(console,core): enable configure on nameIdFormat and encryption #6929

Merged

Conversation

darcyYe
Copy link
Contributor

@darcyYe darcyYe commented Jan 8, 2025

Summary

  1. enable configure on nameIdFormat and encryption through console. (resolves LOG-10673)
  2. configuration on nameIdFormat and encryption can take affect on SAML app flow. (resolves LOG-10671)
  3. enable error message display in the TextArea component.

Testing

Tested locally.
image
image

Checklist

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

Copy link

github-actions bot commented Jan 8, 2025

COMPARE TO master

Total Size Diff 📈 +9.05 KB

Diff by File
Name Diff
packages/console/src/ds-components/Textarea/index.module.scss 📈 +175 Bytes
packages/console/src/ds-components/Textarea/index.tsx 📈 +146 Bytes
packages/console/src/pages/ApplicationDetails/SamlApplicationDetailsContent/Settings.tsx 📈 +4.16 KB
packages/console/src/pages/ApplicationDetails/SamlApplicationDetailsContent/index.module.scss 📈 +102 Bytes
packages/console/src/pages/ApplicationDetails/SamlApplicationDetailsContent/utils.ts 📈 +1.49 KB
packages/core/src/saml-applications/SamlApplication/index.test.ts 📈 +90 Bytes
packages/core/src/saml-applications/SamlApplication/index.ts 📈 +865 Bytes
packages/core/src/saml-applications/SamlApplication/utils.test.ts 📈 +133 Bytes
packages/core/src/saml-applications/SamlApplication/utils.ts 📈 +99 Bytes
packages/core/src/saml-applications/queries/index.ts 📈 +216 Bytes
packages/phrases/src/locales/en/errors/application.ts 📈 +259 Bytes
packages/phrases/src/locales/en/translation/admin-console/application-details.ts 📈 +1.36 KB

@github-actions github-actions bot added feature Cool stuff size/m labels Jan 8, 2025
@darcyYe darcyYe requested a review from simeng-li as a code owner January 8, 2025 09:14
@darcyYe darcyYe changed the title feat(console): enable configure on nameIdFormat and encryption feat(console,core): enable configure on nameIdFormat and encryption Jan 8, 2025
@github-actions github-actions bot added size/l and removed size/m labels Jan 8, 2025
Copy link
Member

@charIeszhao charIeszhao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved to unblock. Please check my comments

@darcyYe darcyYe force-pushed the yemq-enable-configure-on-name-id-format-and-encryption branch from 03b305a to 8178b64 Compare January 8, 2025 10:12
@darcyYe darcyYe force-pushed the yemq-enable-configure-on-name-id-format-and-encryption branch from 8178b64 to a411973 Compare January 8, 2025 10:26
@darcyYe darcyYe merged commit b335ad0 into master Jan 9, 2025
37 checks passed
@darcyYe darcyYe deleted the yemq-enable-configure-on-name-id-format-and-encryption branch January 9, 2025 03:03
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