-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
feat: RLP transactions #567
Conversation
Co-authored-by: smartcontracts <smartcontracts@doge.org>
|
}, | ||
} | ||
} catch (err) { | ||
// Do nothing, fall back to legacy decode. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should increment a Counter here; cc @annieke
We probably don't need this backwards compatibility if we're going to be doing a re-genesis for this upgrade. |
} catch (err) { | ||
// Do nothing, fall back to legacy decode. | ||
} | ||
|
||
try { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't need this decoding anymore because we don't have to be backwards compatible
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Assuming this is added to semver version 0.3.x
Closing this in favor of the |
^ now tracking here |
Co-authored-by: smartcontracts smartcontracts@doge.org
Description
Adds RLP transactions - #496