metaplex-foundation / metaplex-ios

Metaplex Mobile IOS SDK
23 stars 20 forks source link

Create NFT Transaction then sign by Phantom Error #65

Open hstdt opened 1 year ago

hstdt commented 1 year ago

Describe the bug

Based on PhantomDeepLinking, I create NFT Transaction then sign by Phantom, but an error happened - Expected signatures length to be equal to the number of required signatures.

I need some help for fix it. Thank you.

PS:

To Reproduce

Demo Project: MetaplexWallet.zip

See video Below

Expected behavior Sign transaction successfully Screenshots

Smartphone (please complete the following information):

Additional context

PS: init of CreateNftInput should be declare as public.

https://github.com/hstdt/metaplex-ios/blob/fb9a4abdb46263aeeb3373a4f37bc10d4807fc3b/Sources/Metaplex/Modules/NFTS/Operations/CreateNftOnChainOperationHandler.swift#L26-L37

commit: https://github.com/metaplex-foundation/metaplex-ios/commit/fb9a4abdb46263aeeb3373a4f37bc10d4807fc3b

ajamaica commented 1 year ago

Thanks I will take a look on it

ajamaica commented 1 year ago

I found the problem. The reason is that the transaction is not signed. By phantom is just connected. The is a need to use the signer as async signer. This is not done in the code. I can expand the code to make an example