Skip to content
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

fix: upgrade library to validate psbt #264

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

andredevjs
Copy link
Contributor

@andredevjs andredevjs commented Jul 11, 2023

Validate that the signed psbt is valid

look at this inscription, 26482871f33f1051f450f2da9af275794c0b5f1c61ebf35e4467fb42c2813403i0
if the psbt is invalid, we should not show it for buy.
In this branch, it shouldn't be show as listed.

Screenshot 2023-07-11 at 17 10 01

With fix:
Screenshot 2023-07-11 at 17 35 43

@vercel
Copy link

vercel bot commented Jul 11, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nosft ✅ Ready (Inspect) Visit Preview Jul 11, 2023 11:09pm

@netlify
Copy link

netlify bot commented Jul 11, 2023

Deploy Preview for fabulous-meringue-01474b ready!

Name Link
🔨 Latest commit b7c1623
🔍 Latest deploy log https://app.netlify.com/sites/fabulous-meringue-01474b/deploys/64ade12890ee8f0008e81c46
😎 Deploy Preview https://deploy-preview-264--fabulous-meringue-01474b.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@andredevjs andredevjs marked this pull request as draft July 11, 2023 23:37
@andredevjs
Copy link
Contributor Author

This won't work yet, since it throws

Screenshot 2023-07-11 at 17 45 06

@andredevjs andredevjs added the help wanted Extra attention is needed label Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant