-
Notifications
You must be signed in to change notification settings - Fork 6
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
Support partially-signed trasnactions #92
Comments
This also makes the transaction read only... if it's changed the signature is no longer valid |
An implication of that is that you don't need to accurately regenerate the serialised version - it must already exist |
just thinking the partial-signed transaction will expire in 150 blocks (approximately 1 min after it is generated), should there be something to indicate the user the expiry time of the transaction? This is so the user is not surprised when transaction expires if they read the instructions too long. |
Hmmm really good point. I guess this should really be for nonce
transactions. Damn.
On Tue, 25 Oct 2022 at 11:22, Joseph ***@***.***> wrote:
just thinking the partial-signed transaction will expire in 150 blocks
(approximately 1 min after it is generated), should there be something to
indicate the user the expiry time of the transaction? This is so the user
is not surprised when transaction expires if they read the instructions too
long.
—
Reply to this email directly, view it on GitHub
<#92 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACVTZ32ILJGQUSXUKRJOUDWE4R43ANCNFSM6AAAAAARNORAUE>
.
You are receiving this because you commented.Message ID:
***@***.***>
--
Sent from Gmail Mobile
|
For this to work, should be able to:
The text was updated successfully, but these errors were encountered: