Skip to content

feat: add support for Liquidity Hub transactions#5719

Open
therealemjy wants to merge 1 commit into
feat/liquidity-hub-market-page-infofrom
feat/liquidity-hub-transactions
Open

feat: add support for Liquidity Hub transactions#5719
therealemjy wants to merge 1 commit into
feat/liquidity-hub-market-page-infofrom
feat/liquidity-hub-transactions

Conversation

@therealemjy

Copy link
Copy Markdown
Member

Jira ticket(s)

VPD-1562

Changes

  • support Liquidity Hub supply and withdrawal transactions
  • display Hub transaction amounts, values, and labels
  • add Liquidity Hub assets to the transaction source filter

@therealemjy
therealemjy requested a review from cuzz-venus July 23, 2026 14:34
@changeset-bot

changeset-bot Bot commented Jul 23, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 85f5dc3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@venusprotocol/evm Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercel Bot commented Jul 23, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dapp-preview Ready Ready Preview Jul 23, 2026 3:27pm
dapp-testnet Ready Ready Preview Jul 23, 2026 3:27pm
venus.io Ready Ready Preview Jul 23, 2026 3:27pm

Request Review

@greptile-apps

greptile-apps Bot commented Jul 23, 2026

Copy link
Copy Markdown

Greptile Summary

Adds Liquidity Hub supply and withdrawal transactions to account history.

  • Formats Hub transaction amounts, values, labels, and event details.
  • Adds Liquidity Hub assets to the transaction source filter.
  • Extends transaction types, translations, mocks, and tests for Hub activity.

Confidence Score: 5/5

The PR appears safe to merge because no eligible blocking failure remains in this follow-up review.

No blocking failure remains.

Important Files Changed

Filename Overview
apps/evm/src/clients/api/queries/getAccountTransactionHistory/formatApiTransaction/formatToLiquidityHubTransaction/index.ts Adds formatting of API supply and withdrawal records into Liquidity Hub transactions.
apps/evm/src/clients/api/queries/getAccountTransactionHistory/useGetAccountTransactionHistory.ts Supplies Liquidity Hub metadata to account-history formatting.
apps/evm/src/pages/Dashboard/Transactions/index.tsx Adds feature-gated Liquidity Hub assets to the transaction source selector.
apps/evm/src/components/TransactionsList/TransactionRow/Events/index.tsx Routes Liquidity Hub transactions to their dedicated event formatter.
apps/evm/src/types/index.ts Extends the transaction model with Liquidity Hub transaction types and metadata.

Reviews (2): Last reviewed commit: "feat: add support for Liquidity Hub tran..." | Re-trigger Greptile

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: da1d48fa73

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Coverage Report for ./apps/evm

Status Category Percentage Covered / Total
🔵 Lines 83.33% 53133 / 63758
🔵 Statements 83.33% 53133 / 63758
🔵 Functions 63.92% 716 / 1120
🔵 Branches 72.92% 6118 / 8390
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
apps/evm/src/clients/api/queries/getAccountTransactionHistory/index.ts 97.43% 75% 100% 97.43% 1, 47
apps/evm/src/clients/api/queries/getAccountTransactionHistory/useGetAccountTransactionHistory.ts 100% 100% 100% 100%
apps/evm/src/clients/api/queries/getAccountTransactionHistory/formatApiTransaction/index.ts 97.43% 75% 100% 97.43% 1
apps/evm/src/clients/api/queries/getAccountTransactionHistory/formatApiTransaction/formatToLiquidityHubTransaction/index.ts 96.49% 50% 100% 96.49% 2, 63
apps/evm/src/components/TransactionsList/TransactionRow/index.tsx 97.87% 0% 0% 97.87% 37
apps/evm/src/components/TransactionsList/TransactionRow/Events/index.tsx 92% 50% 100% 92% 26-28
apps/evm/src/components/TransactionsList/TransactionRow/Events/formatToLiquidityHubTxEvents/index.ts 96.15% 0% 100% 96.15% 3
apps/evm/src/constants/marketTxTypes.ts 96.15% 0% 100% 96.15% 1
apps/evm/src/pages/Dashboard/Transactions/index.tsx 84.39% 66.66% 0% 84.39% 51-55, 59-65, 67-69, 75-81, 89-96, 115, 198-202, 250
apps/evm/src/types/index.ts 100% 100% 100% 100%
apps/evm/src/utilities/getTransactionName/index.ts 64.1% 55% 100% 64.1% 30-33, 35, 37-45
Generated in workflow #13918 for commit 0bc788a by the Vitest Coverage Report Action

@therealemjy
therealemjy force-pushed the feat/liquidity-hub-market-page-info branch from 3800888 to 6e177cc Compare July 23, 2026 15:18
@therealemjy
therealemjy force-pushed the feat/liquidity-hub-transactions branch from fd4f28c to 85f5dc3 Compare July 23, 2026 15:19
@therealemjy

Copy link
Copy Markdown
Member Author

@greptile

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