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

feat: add hdwallet-metamask-shapeshift-multichain adapter #628

Merged
merged 175 commits into from
Sep 28, 2023

Conversation

0xdef1cafe
Copy link
Collaborator

@0xdef1cafe 0xdef1cafe commented Aug 25, 2023

Disregard the below, this is now published as alpha: https://www.npmjs.com/package/@shapeshiftoss/metamask-snaps-adapter

Note, we had to bump tsc here to accomodate for the version in metamask-snaps monorepo (which we can't avoid because of new TS features from TS 4.4+ that need to be supported in some snaps transient dependencies), which subsequently broke a lot of things at type-level, explaining the disgusting amount of ts-ignores in this diff.

Opening draft to visualize the diff
Note, both shapeshift/metamask-snaps#100, #628, and shapeshift/web#5166 assume verdaccio running on default port 4873, and all packages published there:

image

This is only for development purposes, and config should be reverted to default uplink before merging each of these PRs.

@vercel
Copy link

vercel bot commented Aug 25, 2023

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

Name Status Preview Comments Updated (UTC)
hdwallet ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 26, 2023 6:43pm

@kaladinlight kaladinlight merged commit a353170 into master Sep 28, 2023
4 checks passed
@kaladinlight kaladinlight deleted the metamask-snaps-integration branch September 28, 2023 15:42
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.

4 participants