Skip to content

chore(clerk-js): Remove experimental references to passkeys#3233

Merged
panteliselef merged 4 commits intomainfrom
elef/sdk-1579-remove-experimental-passkey-leftovers
Apr 23, 2024
Merged

chore(clerk-js): Remove experimental references to passkeys#3233
panteliselef merged 4 commits intomainfrom
elef/sdk-1579-remove-experimental-passkey-leftovers

Conversation

@panteliselef
Copy link
Copy Markdown
Contributor

@panteliselef panteliselef commented Apr 19, 2024

Description

Each experimental public api that is dropped in this PR was already replaced with a "stable" one by #3134

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:

@panteliselef panteliselef requested a review from a team April 19, 2024 14:45
@panteliselef panteliselef self-assigned this Apr 19, 2024
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 19, 2024

🦋 Changeset detected

Latest commit: ef08c93

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

This PR includes changesets to release 13 packages
Name Type
@clerk/localizations Minor
@clerk/clerk-js Minor
@clerk/types Minor
@clerk/chrome-extension Patch
@clerk/clerk-expo Patch
@clerk/backend Patch
@clerk/express Patch
@clerk/fastify Patch
@clerk/nextjs Patch
@clerk/clerk-react Patch
@clerk/remix Patch
@clerk/clerk-sdk-node Patch
@clerk/themes 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

Copy link
Copy Markdown
Contributor

@LekoArts LekoArts left a comment

Choose a reason for hiding this comment

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

Small comment, rest LGTM

Comment thread .changeset/tiny-taxis-scream.md Outdated
'@clerk/types': minor
---

Remove any public api that is marked as experimental or has the `__experimental_` prefix that is related to passkeys.
Copy link
Copy Markdown
Contributor

@LekoArts LekoArts Apr 22, 2024

Choose a reason for hiding this comment

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

Suggested change
Remove any public api that is marked as experimental or has the `__experimental_` prefix that is related to passkeys.
Remove experimental Passkeys APIs. This includes any API that is marked as experimental or has the `__experimental_` prefix.
This prepares the Passkeys release to move further along towards a beta release and eventual stable release.

Reads clearer to me to move the passkeys mention to the beginning

@panteliselef panteliselef enabled auto-merge (squash) April 23, 2024 06:47
@panteliselef panteliselef merged commit 956d879 into main Apr 23, 2024
@panteliselef panteliselef deleted the elef/sdk-1579-remove-experimental-passkey-leftovers branch April 23, 2024 06:54
@clerk-cookie clerk-cookie mentioned this pull request Apr 24, 2024
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.

3 participants