Skip to content

Conversation

@LauraBeatris
Copy link
Member

Description

Updates SAML_IDPS to mention Microsoft Entra ID instead of Azure AD.

Microsoft Entra ID is the new name for Azure AD. The names Azure Active Directory, Azure AD, and AAD are replaced with Microsoft Entra ID.

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 Aug 28, 2024
@changeset-bot
Copy link

changeset-bot bot commented Aug 28, 2024

🦋 Changeset detected

Latest commit: 9eb61c7

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

This PR includes changesets to release 18 packages
Name Type
@clerk/types Patch
@clerk/astro Patch
@clerk/backend Patch
@clerk/clerk-js Patch
@clerk/elements Patch
@clerk/clerk-expo 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/chrome-extension 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 requested a review from a team August 28, 2024 14:52
@LauraBeatris LauraBeatris changed the title chore(types): Update SAML_IDPS for Microsoft Entra ID chore(types): Update SAML_IDPS constant for Microsoft Entra ID Aug 28, 2024
Copy link
Member

@panteliselef panteliselef left a comment

Choose a reason for hiding this comment

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

I'm not familiar with the branding/terminology, feel free to ignore my comment

},
saml_microsoft: {
name: 'Microsoft Azure AD',
name: 'Microsoft Entra ID (Formerly AD)',
Copy link
Member

Choose a reason for hiding this comment

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

🙃

Suggested change
name: 'Microsoft Entra ID (Formerly AD)',
name: 'Microsoft Entra ID (Azure AD)',

Copy link
Member Author

Choose a reason for hiding this comment

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

I tend to prefer Formerly since it denotes that AD is deprecated.

@LauraBeatris LauraBeatris merged commit 1369301 into main Aug 28, 2024
@LauraBeatris LauraBeatris deleted the update-saml-idps-type branch August 28, 2024 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants