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

Display account token transfers #624

Merged
merged 5 commits into from
Jun 30, 2023

Conversation

csillag
Copy link
Contributor

@csillag csillag commented Jun 29, 2023

This is built on top of #616, so only the last few comments are relevant to this task.

Display token transfers related to an account.

Design:

Can be tested (for example) with 0x643c8A8120eFB675E82C5C58d8D473f5aDd0B5e6.

Unfortunately, our data requirements are not completely clear at this point, so only a part of the visual is implemented. However, this is probably all we can do before the launch, so it makes sense to review this now.

@github-actions
Copy link

github-actions bot commented Jun 29, 2023

Deployed to Cloudflare Pages

Latest commit: 36f4f8df5c5113fbeaddd91973183a748cb84cb8
Status:✅ Deploy successful!
Preview URL: https://bb3f962e.oasis-explorer.pages.dev

@csillag csillag changed the title Csillag/display account token transfers Display account token transfers Jun 29, 2023
@csillag csillag force-pushed the csillag/display-account-token-transfers branch 7 times, most recently from 7b31e2c to 6d58e95 Compare June 29, 2023 17:00
@csillag csillag marked this pull request as ready for review June 29, 2023 17:20
@csillag csillag requested review from lukaw3d and buberdds and removed request for lukaw3d June 29, 2023 17:20
Copy link
Member

@lukaw3d lukaw3d left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

src/app/components/Tokens/TokenTransfers.tsx Outdated Show resolved Hide resolved
src/routes.tsx Show resolved Hide resolved
src/app/components/Tokens/TokenTransfers.tsx Outdated Show resolved Hide resolved
src/app/pages/AccountDetailsPage/hook.ts Show resolved Hide resolved
@csillag csillag force-pushed the csillag/display-account-token-transfers branch 4 times, most recently from 5273e48 to 680f5f5 Compare June 30, 2023 01:38
@csillag csillag requested a review from lukaw3d June 30, 2023 11:05
@csillag csillag force-pushed the csillag/display-account-token-transfers branch 4 times, most recently from db08bd4 to 5398e1d Compare June 30, 2023 13:34
(For ticker and decimals to be used
when displaying token transfers in a list.)
@csillag csillag force-pushed the csillag/display-account-token-transfers branch from 5398e1d to 36f4f8d Compare June 30, 2023 13:35
@csillag csillag merged commit b3858d1 into master Jun 30, 2023
7 checks passed
@csillag csillag deleted the csillag/display-account-token-transfers branch June 30, 2023 13:37
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.

None yet

3 participants