Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Smart Wallet SDK: use API #966

Merged
merged 97 commits into from
Feb 25, 2025
Merged

Smart Wallet SDK: use API #966

merged 97 commits into from
Feb 25, 2025

Conversation

Destiner
Copy link
Contributor

@Destiner Destiner commented Feb 25, 2025

Description

Updates the Smart Wallet SDK to use Wallet API.

Keeps the SDK surface almost exact as it was before, but removes some minor functionality that we can't replicate with server wallets.

Also introduces high-level docs and a migration plan via README.md.

Full context: https://linear.app/crossmint/project/api-powered-sdk-e873974b18ca/issues

Recreation of #913 (previous PR was pushed from the fork which didn't trigger CI properly)

Note: This was mostly written from scratch, and it's best to review it from scratch instead of going through the diffs.

Test plan

  • Create a client-side API key with the necessary scopes
  • Get the JWT using Crossmint Auth
  • Call the API endpoints via SDK

Criteria:

  • Functionality: A simple web app that uses SDK should work properly
  • No breaking changes: The playground should work with minimal changes
  • Data sync: A wallet created with SDK 0.1.x should work with SDK 0.2.x

Package updates

  • @crossmint/client-sdk-smart-wallet: 0.1.300.2.0
  • @crossmint/client-sdk-react-ui: 0.1.300.2.0

Destiner and others added 30 commits February 25, 2025 11:18
* remove all code
* use API
* wallet creation & tx sending for ECDSA signers
* clean up dependencies
* auth user type: twitter

* changeset
* wallet creation
* transaction sending flow
* `signMessage`
* `signTypedData`
…uth app (#912)

* Adapts server sdk to non-nodejs envs and adds example to next auth app with middleware

* pnpm lock
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
jmderby and others added 19 commits February 25, 2025 11:18
* Fix: when we show continue w/ google btn

* added changeset
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* Fix: Google icon size on initial page load

* added changeset
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* update nanoid

* add changeset
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Copy link

changeset-bot bot commented Feb 25, 2025

🦋 Changeset detected

Latest commit: b4ef193

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

This PR includes changesets to release 8 packages
Name Type
@crossmint/client-sdk-smart-wallet Minor
@crossmint/client-sdk-react-ui Minor
@crossmint/client-sdk-vue-ui-starter Patch
@crossmint/client-sdk-smart-wallet-react-starter Patch
@crossmint/auth-ssr-nextjs-demo Patch
@crossmint/client-sdk-react-ui-starter Patch
@crossmint/client-sdk-nextjs-starter Patch
@crossmint/client-sdk-smart-wallet-next-starter 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

vercel bot commented Feb 25, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
crossmint-sdk-remix-ssr ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 25, 2025 11:38am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
smart-wallet-auth-demo ⬜️ Ignored (Inspect) Visit Preview Feb 25, 2025 11:38am

@Destiner Destiner merged commit 98d8197 into main Feb 25, 2025
5 checks passed
@Destiner Destiner deleted the timur/smart-wallet-sdk-use-api branch February 25, 2025 15:23
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.

8 participants