Skip to content
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

Merged
merged 46 commits into from
Nov 9, 2023
Merged

feat: add amendment info #135

merged 46 commits into from
Nov 9, 2023

Conversation

pdp2121
Copy link
Collaborator

@pdp2121 pdp2121 commented Sep 20, 2023

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

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactor (non-breaking change that only restructures code)
  • Tests (You added tests for code that already exists, or your new feature included in this PR)
  • Documentation Updates
  • Release

package.json Show resolved Hide resolved
src/api/routes/v1/index.ts Outdated Show resolved Hide resolved
src/api/routes/v1/info.ts Outdated Show resolved Hide resolved
Copy link

@jonathanlei jonathanlei left a 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

@pdp2121 pdp2121 requested a review from mvadari November 2, 2023 01:59
Comment on lines +94 to +95
const response = await axios.get(
'https://raw.githubusercontent.com/XRPLF/xrpl-dev-portal/master/content/resources/known-amendments.md',
Copy link
Collaborator

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.

Copy link
Collaborator Author

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?

Copy link
Collaborator

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.

src/shared/types/index.ts Outdated Show resolved Hide resolved
ARCHITECTURE.md Outdated Show resolved Hide resolved
@pdp2121 pdp2121 requested a review from mvadari November 7, 2023 23:41
@pdp2121 pdp2121 merged commit cb10166 into main Nov 9, 2023
@pdp2121 pdp2121 deleted the add-amendment-info branch November 9, 2023 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants