Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

Update flag to allow output scripts #13

Merged
merged 3 commits into from
Oct 2, 2023
Merged

Update flag to allow output scripts #13

merged 3 commits into from
Oct 2, 2023

Conversation

Imamah-Zafar
Copy link
Contributor

Update the flag according to latest package upgrade of @scure/btc-signer

@Imamah-Zafar Imamah-Zafar requested a review from yknl October 2, 2023 07:28
@yknl
Copy link
Contributor

yknl commented Oct 2, 2023

Should also update the @scure/btc-signer version in package.json

@msxakk89
Copy link

msxakk89 commented Oct 3, 2023

These changes have not resoled the PSBT issues. Using the latest version of the Sats Connect Test App fails to broadcast transaction:

Transaction Failed
Request failed with status code 400

Inspecting raw PSBT transaction shows absence of digital sig:

{
      "txid": "435a438f35d64d27b287d4ba9fdb00911f469f2490cb539bc8f546750c402809",
      "vout": 1,
      "scriptSig": {
        "asm": "",
        "hex": ""
      },
      "sequence": 4294967295
    },
    {
      "txid": "dbcd6eb46be739ec7821d82898a46438020305d529776ef3c4290c63d536fc2c",
      "vout": 1,
      "scriptSig": {
        "asm": "",
        "hex": ""
      },
      "sequence": 4294967295
    }
  ],

@DuskaT021 DuskaT021 deleted the imamahzafar/update-flag branch October 26, 2023 09:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants