-
Notifications
You must be signed in to change notification settings - Fork 971
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
Update mobile adapter #882
base: master
Are you sure you want to change the base?
Conversation
…cs/wallet-adapter into update-mobile-adapter
@jordaaash I think I got our libs all fixed up. This branch is currently using @solana-mobile/wallet-adapter-mobile v2.1.0-alpha1 and its building a testing both locally and on CI (I think) I am going to test this alpha build a bit more then publish a proper 2.1.0 release and update this PR |
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@solana-mobile/wallet-adapter-mobile@2.0.1 |
@jsx bump. Do you have any concerns with merging this? |
pnpm-lock.yaml
Outdated
dependencies: | ||
'@wallet-standard/base': 1.0.1 | ||
'@wallet-standard/features': 1.0.3 | ||
dev: false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're resolving multiple versions of some packages now, it might be something on our end. Looking into it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, lmk if I can help
@jordaaash there are a couple remaining dupes:
and
due to Thoughts? |
No description provided.