-
Notifications
You must be signed in to change notification settings - Fork 146
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
Comments
This is just happening on signet? |
This is more common than we thought. I am able to reproduce the issue on mainnet too now. To reproduce See it here (video): |
@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 |
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. 🤔 |
Would be helpful here to know any errors in the console of either ME or the wallet popup. |
I understand now the bug is just the window staying open, not that we are broadcasting the tx. |
One of the devs at Magic Eden reported this.
The text was updated successfully, but these errors were encountered: