-
Notifications
You must be signed in to change notification settings - Fork 9
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: add amendment info #135
Conversation
…ator-history-service into add-amendment-info
Co-authored-by: Mayukha Vadari <mvadari@ripple.com>
Co-authored-by: Caleb Kniffen <ckniffen@ripple.com>
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 other than @mvadari 's typo changes in the REGEX
Co-authored-by: Mayukha Vadari <mvadari@ripple.com>
const response = await axios.get( | ||
'https://raw.githubusercontent.com/XRPLF/xrpl-dev-portal/master/content/resources/known-amendments.md', |
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.
This is unrelated to this PR but I wonder if there's some way that this information can also be fetched from rippled. Seems difficult.
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.
@intelliot @ckniffen any ideas on how we can fetch rippled version of each amendment by id?
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.
The docs is the only place i know of.
High Level Overview of Change
Add a table to store amendments information:
Amendment’s hash
Amendment’s name
Enabled rippled version
This will also provide more stable source of truth for AmendmentEnabled transaction in Explorer, which hits 2 repos every time it was called.
This information will be accessed via
/network/amendments/info
endpoint.Type of Change