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: added ticket_sequence to Transaction class common fields #428
Conversation
nit: this PR should have a name somewhat akin to https://www.conventionalcommits.org/en/v1.0.0/ (see: other PRs) |
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.
LGTM
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.
Oh, could you actually also add this as an entry in HISTORY.md?
Maybe consider adding a link to https://xrpl.org/tickets.html somewhere in here? Just a thought |
Co-authored-by: Elliot Lee <github.public@intelliot.com>
Co-authored-by: Elliot Lee <github.public@intelliot.com>
High Level Overview of Change
Added ticket_sequence field to common Transaction class fields
Context of Change
Based on issue "Add common field ticket_sequence to Transaction #413"
Type of Change
Test Plan
Wrote out unit tests