Skip to content

feat(clerk-js,types): Surface enterprise accounts in UserProfile - #4518

Merged
LauraBeatris merged 21 commits into
mainfrom
laura/orgs-293-display-enterprise-accounts-from-userenterprise_accounts
Nov 13, 2024
Merged

feat(clerk-js,types): Surface enterprise accounts in UserProfile#4518
LauraBeatris merged 21 commits into
mainfrom
laura/orgs-293-display-enterprise-accounts-from-userenterprise_accounts

Conversation

@LauraBeatris

@LauraBeatris LauraBeatris commented Nov 7, 2024

Copy link
Copy Markdown
Contributor

Description

Resolves ORGS-293, ORGS-294

Surfaces user.enterprise_accounts instead of user.saml_accounts. This is going to support multiple enterprise protocols from now on, such as OpenID connect.

Checklist

  • npm test runs as expected.
  • npm run build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

@LauraBeatris LauraBeatris self-assigned this Nov 7, 2024
@changeset-bot

changeset-bot Bot commented Nov 7, 2024

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: e71267b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 21 packages
Name Type
@clerk/clerk-js Minor
@clerk/types Minor
@clerk/chrome-extension Patch
@clerk/clerk-expo Patch
@clerk/astro Patch
@clerk/backend Patch
@clerk/elements Patch
@clerk/expo-passkeys Patch
@clerk/express Patch
@clerk/fastify Patch
@clerk/localizations Patch
@clerk/nextjs Patch
@clerk/clerk-react Patch
@clerk/remix Patch
@clerk/clerk-sdk-node Patch
@clerk/shared Patch
@clerk/tanstack-start Patch
@clerk/testing Patch
@clerk/themes Patch
@clerk/ui Patch
@clerk/vue Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@LauraBeatris
LauraBeatris force-pushed the laura/orgs-293-display-enterprise-accounts-from-userenterprise_accounts branch 2 times, most recently from 27932f1 to 4b3eb72 Compare November 7, 2024 20:47
@LauraBeatris
LauraBeatris force-pushed the laura/orgs-293-display-enterprise-accounts-from-userenterprise_accounts branch 2 times, most recently from db250d9 to 6a5ddaa Compare November 7, 2024 21:17
@LauraBeatris LauraBeatris changed the title feat(clerk-js,backend,types): Display enterprise accounts on UserProfile feat(clerk-js,types): Display enterprise accounts on UserProfile Nov 7, 2024
@LauraBeatris
LauraBeatris force-pushed the laura/orgs-293-display-enterprise-accounts-from-userenterprise_accounts branch 4 times, most recently from 1815e12 to 16eda49 Compare November 8, 2024 00:24
@LauraBeatris LauraBeatris changed the title feat(clerk-js,types): Display enterprise accounts on UserProfile feat(clerk-js,types): Support multiple enterprise protocols in UserProfile Nov 8, 2024
@LauraBeatris
LauraBeatris force-pushed the laura/orgs-293-display-enterprise-accounts-from-userenterprise_accounts branch from 16eda49 to 094f540 Compare November 8, 2024 00:44
@LauraBeatris
LauraBeatris force-pushed the laura/orgs-293-display-enterprise-accounts-from-userenterprise_accounts branch from 094f540 to e5aa840 Compare November 8, 2024 00:46
Comment thread packages/clerk-js/src/ui/components/UserProfile/EnterpriseAccountsSection.tsx Outdated
Comment thread packages/types/src/appearance.ts Outdated
@LauraBeatris
LauraBeatris force-pushed the laura/orgs-293-display-enterprise-accounts-from-userenterprise_accounts branch from e5aa840 to d270ba9 Compare November 12, 2024 17:39
@LauraBeatris LauraBeatris changed the title feat(clerk-js,types): Support multiple enterprise protocols in UserProfile feat(clerk-js,types): Surface enterprise accounts in UserProfile Nov 12, 2024
@LauraBeatris
LauraBeatris force-pushed the laura/orgs-293-display-enterprise-accounts-from-userenterprise_accounts branch from 1a29195 to e019947 Compare November 12, 2024 18:03
@LauraBeatris
LauraBeatris force-pushed the laura/orgs-293-display-enterprise-accounts-from-userenterprise_accounts branch from e019947 to d21ba70 Compare November 12, 2024 18:03
Comment thread packages/clerk-js/src/ui/components/UserProfile/AccountPage.tsx
@LauraBeatris

Copy link
Copy Markdown
Contributor Author

@NicolasLopes7 Preserving the disableAdditionalIdentifications check:

CleanShot 2024-11-12 at 15 33 01

CleanShot.2024-11-12.at.15.31.01.mp4

@LauraBeatris
LauraBeatris force-pushed the laura/orgs-293-display-enterprise-accounts-from-userenterprise_accounts branch 2 times, most recently from 6751683 to 3a253db Compare November 13, 2024 17:08

@panteliselef panteliselef left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🛳️

Comment thread packages/types/src/appearance.ts
Previously, SAML was the only supported enterprise type, therefore the UI description was only based on `SamlIdpSlug`

It'll now use `EnterpriseProvider` to handle multiple connection types.
Using `user.enterpriseAccounts` to display on `UserProfile`, instead of `user.samlAccounts`

This is going to support the upcoming connection types such as EASIE, OIDC

Also, deletes `useSaml` hook as it was a weak abstraction around SAML constants for logo URLs and naming
Refer to `user.enterprise_accounts` to verify if the user is allowed to introduce additional identifications
@LauraBeatris
LauraBeatris force-pushed the laura/orgs-293-display-enterprise-accounts-from-userenterprise_accounts branch from 3a253db to d4cdd22 Compare November 13, 2024 18:09
@LauraBeatris
LauraBeatris merged commit 1c0b500 into main Nov 13, 2024
@LauraBeatris
LauraBeatris deleted the laura/orgs-293-display-enterprise-accounts-from-userenterprise_accounts branch November 13, 2024 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants