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

After clicking confirm with any PSBT on ME the window stays open yet sends the transaction #4121

Closed
314159265359879 opened this issue Aug 10, 2023 · 7 comments · Fixed by #4152
Assignees
Labels
area:bitcoin area:testnet bug Functionality broken bug-p3 Non-critical functionality broken for many users, or there are clear workarounds

Comments

@314159265359879
Copy link
Contributor

One of the devs at Magic Eden reported this.

image

@markmhendrickson
Copy link
Collaborator

This is just happening on signet?

@314159265359879 314159265359879 changed the title [Signet] after clicking confirm with any PSBT the window stays open yet sends the transaction After clicking confirm with any PSBT on ME the window stays open yet sends the transaction Aug 24, 2023
@314159265359879 314159265359879 added the bug-p3 Non-critical functionality broken for many users, or there are clear workarounds label Aug 24, 2023
@314159265359879
Copy link
Contributor Author

This is more common than we thought. I am able to reproduce the issue on mainnet too now.

To reproduce
Visit magiceden.io
Go to Bitcoin assets
Buy any Ordinal inscription
When you click to confirm the window stays open but the transaction is send.

See it here (video):
https://github.com/hirosystems/wallet/assets/33360391/4e576053-3d6c-4636-a94b-f60df322e1a5

@markmhendrickson
Copy link
Collaborator

@fbwoolf Might the recent broadcasting parameter work have inadvertently caused this for (some) cases in which the app does not request broadcast?

@fbwoolf
Copy link
Contributor

fbwoolf commented Aug 25, 2023

@fbwoolf Might the recent broadcasting parameter work have inadvertently caused this for (some) cases in which the app does not request broadcast?

The video shows a notification of theirs, not sure how that would be triggered on our end? The broadcast parameter isn't required and it is checked here for the broadcast to occur: https://github.com/hirosystems/wallet/blob/99539d9af0e4c162802fe5efd3f741fcd893422b/src/app/pages/rpc-sign-psbt/use-rpc-sign-psbt.tsx#L112

@fbwoolf
Copy link
Contributor

fbwoolf commented Aug 25, 2023

Also, is ME using the rpc request? Bc this was only added to the rpc request.

EDIT: It seems odd to me no one else is reporting if on our end? When we broadcast it, we navigate to either the tx summary or an error page. 🤔

@fbwoolf
Copy link
Contributor

fbwoolf commented Aug 25, 2023

Would be helpful here to know any errors in the console of either ME or the wallet popup.

@fbwoolf
Copy link
Contributor

fbwoolf commented Aug 26, 2023

I understand now the bug is just the window staying open, not that we are broadcasting the tx.

@markmhendrickson markmhendrickson moved this from Assigned to Triage in Hiro Wallet (DEPRECATED) Aug 28, 2023
@fbwoolf fbwoolf moved this from Triage to Assigned in Hiro Wallet (DEPRECATED) Aug 29, 2023
@fbwoolf fbwoolf moved this from Assigned to WIP in Hiro Wallet (DEPRECATED) Aug 30, 2023
@fbwoolf fbwoolf linked a pull request Aug 30, 2023 that will close this issue
@fbwoolf fbwoolf moved this from WIP to Review in Hiro Wallet (DEPRECATED) Aug 30, 2023
@fbwoolf fbwoolf moved this from Review to Released or closed in Hiro Wallet (DEPRECATED) Aug 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:bitcoin area:testnet bug Functionality broken bug-p3 Non-critical functionality broken for many users, or there are clear workarounds
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants