mlabs-haskell / seabug-contracts

3 stars 1 forks source link

Update minting contract to use `mustPayToPubKeyAddress` #29

Closed rynoV closed 2 years ago

rynoV commented 2 years ago

Currently (#22) there's a warning:

  in module Seabug.Contract.CnftMint
  at src/Seabug/Contract/CnftMint.purs:33:1 - 36:46 (line 33, column 1 - line 36, column 46)

    A custom warning occurred while solving type class constraints:

      Some wallets may not recognize addresses without a staking key component. Consider usingmustPayToPubKeyAddress

  in value declaration mintCnft

Don't think this is critical because the minting is working. It should be a quick fix, I just couldn't test it because Nami and Gero weren't working and my KeyWallet didn't have a staking key. The fix should be:

rynoV commented 2 years ago

Update Mint to search for utxos at payPubKeyHashBaseAddress instead of payPubKeyHashEnterpriseAddress

Due to CTL's balancing, this isn't strictly necessary as the utxos get pulled in automatically, but I think it's best to be explicit about where we expect the CNFT to be.