Skip to content

feat(clerk-js): Experimental CRUD methods for passkeys#2926

Merged
panteliselef merged 1 commit intomainfrom
elef/sdk-1439-list-update-passkeys
Mar 5, 2024
Merged

feat(clerk-js): Experimental CRUD methods for passkeys#2926
panteliselef merged 1 commit intomainfrom
elef/sdk-1439-list-update-passkeys

Conversation

@panteliselef
Copy link
Copy Markdown
Contributor

@panteliselef panteliselef commented Mar 4, 2024

Description

Experimental support for reading, updating, and deleting a user's registered passkeys.

  • Get the user's passkeys
    clerk.user.__experimental__passkeys
  • Update the name of a passkey
    clerk.user.__experimental__passkeys?.[0].update({name:'work laptop passkey'})
  • Delete a passkey
    clerk.user.__experimental__passkeys?.[0].delete()

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 self-assigned this Mar 4, 2024
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Mar 4, 2024

🦋 Changeset detected

Latest commit: a4c0e0a

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

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

Comment on lines -46 to -52
/**
* TODO-PASSKEYS: Implement this later
*
* GET /v1/me/passkeys
*/
static async get() {}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Instead we grab the passkeys directly from the User resource without performing a http request.

@panteliselef panteliselef requested review from a team, nikosdouvlis and tmilewski and removed request for a team and tmilewski March 4, 2024 12:27
@panteliselef panteliselef added this pull request to the merge queue Mar 5, 2024
Merged via the queue into main with commit 008ac42 Mar 5, 2024
@panteliselef panteliselef deleted the elef/sdk-1439-list-update-passkeys branch March 5, 2024 18:08
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