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 pagination to addresses #322

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

jgryffindor
Copy link
Contributor

@jgryffindor jgryffindor commented Sep 23, 2024

For addresses with larger numbers of transactions, we were missing pagination on the /addresses/[address] page in the UI. This branch adds pagination to the endpoint in the server and to the UI.

Description

  • Added pagination and interface definition to address service function for findOne address.
  • Added pagination to addresses controller endpoint for one address.
  • Added pager component to UI for addresses page and added pagination meta to API query in frontend.
  • Fixed migrations endpoint to display most recent migrations first
  • Fixed tokens query to retrieve 100 tokens, which was limited to 25 and missing tokens from the alpha prod network.

Related Issue

Fixes # (issue)
#318
#320
#306

Testing

Tested locally against a clone of prod database.

Breaking Changes (if applicable)

None

Screenshots (if applicable)

Pager on Address transactions view
image

Fix of precision on ledger-send:

image

Checklist:

  • I have read and followed the CONTRIBUTING guidelines for this project.
  • I have added or updated tests and they pass.
  • I have added or updated documentation and it is accurate.
  • I have noted any breaking changes in this module or downstream modules.

Added pagination and interface definition to addresses service function for findOne address.
Added pagination to addresses controller endpoint for one address.
Added pager component to UI for addresses page and added pagination meta to api query.
Fix for tokensQuery not returning all tokens and causing Unknown Tokens across many denoms.
Retrieve the first page of 100 tokens (need to refactor later to add pagination).
Standardize precision conversion in mint-burn to ledger-send format.
Set precision to token precision in ledger-send, convert to string then use parseFloat to remove trailing 0s.
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.

1 participant