Skip to content

feat(internet-identity): update for @icp-sdk/auth v6 API#181

Closed
sea-snake wants to merge 1 commit intodfinity:mainfrom
sea-snake:update-internet-identity-authclient-v6
Closed

feat(internet-identity): update for @icp-sdk/auth v6 API#181
sea-snake wants to merge 1 commit intodfinity:mainfrom
sea-snake:update-internet-identity-authclient-v6

Conversation

@sea-snake
Copy link
Copy Markdown

Summary

Updates the internet-identity skill to match the new @icp-sdk/auth v6 API.

  • AuthClient.create() becomes new AuthClient(options); identityProvider is now passed to the constructor.
  • Callback-based login({ onSuccess, onError }) becomes promise-based await signIn({ maxTimeToLive }) returning the Identity directly.
  • isAuthenticated() is now sync; getIdentity() is now async.
  • Pitfall Migrate @dfinity/* packages to @icp-sdk/* #3 reworded around the new promise-based flow; pitfall Improve repo structure, CI pipeline, and contributor experience #5 updated to mention the new failure modes.
  • Code example rewritten end-to-end against the new API. Adds a comment about openIdProvider for one-click sign-in.
  • Bumps prerequisite to @icp-sdk/auth >= 6.0.0.

npm run validate passes. Companion PR: dfinity/developer-docs#189.

@sea-snake sea-snake requested review from a team and JoshDFN as code owners May 4, 2026 15:49
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 4, 2026

Thank you for contributing! Unfortunately this repository does not accept external contributions yet.

We are working on enabling this by aligning our internal processes and our CI setup to handle external contributions. However this will take some time to set up so in the meantime we unfortunately have to close this Pull Request.

We hope you understand and will come back once we accept external PRs.

— The DFINITY Foundation

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant