Skip to content

feat(ui-ux): api integration for transactions list and details page#151

Merged
pierregee merged 28 commits into
mainfrom
lyka/transactions
May 22, 2023
Merged

feat(ui-ux): api integration for transactions list and details page#151
pierregee merged 28 commits into
mainfrom
lyka/transactions

Conversation

@lykalabrada

@lykalabrada lykalabrada commented May 1, 2023

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:

  • Integrate transactions list data from API
  • Integrate transaction detail data from API

Which issue(s) does this PR fixes?:

Fixes #

Additional comments?:

Developer Checklist:

  • Read your code changes at least once
  • Your UI implementation visually matched the rendered design*
  • Unit tests*
  • Added e2e tests*

@netlify

netlify Bot commented May 1, 2023

Copy link
Copy Markdown

Deploy Preview for defimetascan failed.

Name Link
🔨 Latest commit 48bf8f1
🔍 Latest deploy log https://app.netlify.com/sites/defimetascan/deploys/646adf6481391700081f9a5a

@lykalabrada lykalabrada changed the title Lyka/transactions feat(ui-ux): integrate api for transactions list and details page May 1, 2023
@lykalabrada lykalabrada force-pushed the lyka/transactions branch from 871d79f to 72a2cc1 Compare May 1, 2023 13:29
@lykalabrada lykalabrada marked this pull request as ready for review May 1, 2023 13:32
@lykalabrada lykalabrada marked this pull request as draft May 1, 2023 13:32
@lykalabrada lykalabrada changed the title feat(ui-ux): integrate api for transactions list and details page feat(ui-ux): api integration for transactions list and details page May 1, 2023
@github-actions github-actions Bot added the kind/feature New feature request label May 1, 2023
@lykalabrada lykalabrada force-pushed the lyka/transactions branch from 72a2cc1 to d524886 Compare May 1, 2023 13:34
@lykalabrada lykalabrada self-assigned this May 2, 2023
Comment thread src/pages/block/[id].tsx
@pierregee

Copy link
Copy Markdown
Contributor
  1. I added the pagination
  2. Fixed UTC format for month
  3. Get API url based on network

@pierregee

Copy link
Copy Markdown
Contributor

Let's update the tests in another PR

@pierregee pierregee marked this pull request as ready for review May 22, 2023 03:27
@pierregee pierregee merged commit 4ccf380 into main May 22, 2023
@pierregee pierregee deleted the lyka/transactions branch May 22, 2023 03:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/feature New feature request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants