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: get transaction history from API #133

Merged
merged 40 commits into from
Nov 23, 2023
Merged

Conversation

Abhikumar98
Copy link
Contributor

No description provided.

@swarmia
Copy link

swarmia bot commented Sep 21, 2023

@Abhikumar98 Abhikumar98 changed the title Lf 4968 tx history endpoint chore: get transaction history from backend Sep 21, 2023
packages/widget/src/hooks/useTransactionHistory.ts Outdated Show resolved Hide resolved
packages/widget/src/hooks/useTransactionHistory.ts Outdated Show resolved Hide resolved
packages/widget/src/hooks/useTransactionHistory.ts Outdated Show resolved Hide resolved
packages/widget/src/hooks/useTransactionHistory.ts Outdated Show resolved Hide resolved
Comment on lines 1 to 9
import { List } from '@mui/material';
import { useVirtualizer } from '@tanstack/react-virtual';
import type { FC, MutableRefObject } from 'react';
import { useEffect } from 'react';

import type { FullStatusData, StatusResponse } from '@lifi/sdk';

import { TransactionHistoryItem } from './TransactionHistoryItem';
import { TokenListItemSkeleton } from '../../components/TokenList/TokenListItem';
Copy link
Member

Choose a reason for hiding this comment

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

We need to start enforcing import sorting on save 😄

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes we should, if you already have a setting enabled for this in VS Code let's push it to .vscode so that linting remains consistent

packages/widget/src/utils/converters.ts Outdated Show resolved Hide resolved
packages/widget/src/utils/converters.ts Outdated Show resolved Hide resolved
packages/widget/src/utils/converters.ts Outdated Show resolved Hide resolved
@chybisov chybisov changed the title chore: get transaction history from backend feat: get transaction history from backend Sep 25, 2023
@Abhikumar98
Copy link
Contributor Author

CleanShot.2023-09-26.at.20.18.35.mp4

Added more skeleton to show better loaders for data loading.

There is a delay upon fetching the transaction details immediately post completing a transaction. Because the whole history is being fetched again.

Attempting to shorten this delay by fetching and appending the latest transaction using getStatus instead of fetching the whole history.

@github-actions
Copy link

github-actions bot commented Oct 4, 2023

Hey! This is your new endopint: https://f3bb1a97.widget-lf4968txhi.pages.dev

@github-actions
Copy link

Hey! This is your new endopint: https://ac8e1f0f.widget-lf4968txhi.pages.dev

@Abhikumar98 Abhikumar98 added wait Wait. and removed testing labels Oct 17, 2023
@Abhikumar98
Copy link
Contributor Author

Abhikumar98 commented Oct 17, 2023

This PR is on hold due to Backend's Pagination task

The frontend is completed yet, waiting for backend to finish up with the pagination implementation and then the frontend would continue implementing this

@chybisov chybisov added the WIP Work in progress label Nov 16, 2023
@chybisov chybisov added testing and removed WIP Work in progress wait Wait. labels Nov 22, 2023
@chybisov chybisov self-assigned this Nov 22, 2023
Copy link

Hey! This is your new endopint: https://ab6b623f.widget-lf4968txhi.pages.dev

Copy link

Hey! This is your new endopint: https://7be5126f.widget-lf4968txhi.pages.dev

@chybisov chybisov changed the title feat: get transaction history from backend feat: get transaction history from API Nov 23, 2023
@chybisov chybisov merged commit 1da1acc into main Nov 23, 2023
1 check passed
@chybisov chybisov deleted the lf-4968-tx-history-endpoint branch November 23, 2023 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants