-
Notifications
You must be signed in to change notification settings - Fork 116
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
Wc 2 #3045
Wc 2 #3045
Conversation
A new deploy preview is available on Cloudflare Pages at https://e146b661.taquito-test-dapp.pages.dev |
New packages have been deployed to the preview repository at https://npm.preview.tezostaquito.io/. Published packages:
|
Deploying taquito with Cloudflare Pages
|
this.getSession(); | ||
throw new ActiveAccountUnspecified(); | ||
} | ||
return this.getSession().self.publicKey; |
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.
@hui-an-yang
When looking at the session returned by Kukai session.self.publicKey
is not the Tezos public key of the active account:
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.
thank you will updated the logic
Added support to sign payload in the test-dapp using wallet connect, fix getPk and sign in the wallet connect package
feat(wallet connect): sign payload using wc in test-dapp
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.
Looks good to me !
fee?: string; | ||
} | ||
|
||
export interface TransferParams |
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.
Wondering if it would make sense to move those types so we can use them in beacon package too
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.
@roxaneletourneau Thank you, that'll be a great idea but it seems @airgap/beacon-types has their own type PartialTezosOperation
and our walletAPI has type WalletParamsWithKind
.
This also made me realize that even in walletconnect package supports all contractAPI operations but walletAPI doesn't support them that their use will be limited.
I'll merge this PR for now and start a new pr to support all contractAPI ops in walletAPI and consolidate the types
Thank you for your contribution to Taquito.
Before submitting this PR, please make sure:
In this PR, please also make sure:
closes #TICKETNUMBER
in the description box (when applicable)Release Note Draft Snippet
If relevant, please write a summary of your change that will be suitable for
inclusion in the Release Notes for the next Taquito release.