Skip to content

feat: connect Liquidity Hub feature to API - #5757

Open
therealemjy wants to merge 8 commits into
mainfrom
feat/liquidity-hub-api
Open

feat: connect Liquidity Hub feature to API#5757
therealemjy wants to merge 8 commits into
mainfrom
feat/liquidity-hub-api

Conversation

@therealemjy

Copy link
Copy Markdown
Member

Jira ticket(s)

VPD-1564

Changes

  • connect Liquidity Hub feature to API

@vercel

vercel Bot commented Jul 30, 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 Aug 18, 2026 9:30am
dapp-testnet Ready Ready Preview Aug 18, 2026 9:30am
venus.io Ready Ready Preview Aug 18, 2026 9:30am

Request Review

@changeset-bot

changeset-bot Bot commented Jul 30, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8ed234b

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

@therealemjy
therealemjy force-pushed the feat/liquidity-hub-api branch from 84c5ae5 to 306bc8c Compare July 30, 2026 16:09
@therealemjy
therealemjy force-pushed the feat/liquidity-hub-api branch from 306bc8c to bcd63ab Compare July 30, 2026 16:33
@therealemjy
therealemjy force-pushed the feat/liquidity-hub-api branch from bcd63ab to 77ad3a3 Compare July 30, 2026 17:02
@therealemjy
therealemjy force-pushed the feat/liquidity-hub-api branch from 77ad3a3 to 5d46684 Compare July 30, 2026 17:20
@therealemjy
therealemjy force-pushed the feat/liquidity-hub-api branch from 5d46684 to 9a40087 Compare July 31, 2026 07:46
@therealemjy
therealemjy force-pushed the feat/liquidity-hub-api branch from 9a40087 to afe4de8 Compare July 31, 2026 08:45
@therealemjy
therealemjy force-pushed the feat/liquidity-hub-api branch from afe4de8 to cd9ac1d Compare July 31, 2026 12:53
@greptile-apps

greptile-apps Bot commented Aug 10, 2026

Copy link
Copy Markdown

Greptile Summary

The PR replaces mock-backed Liquidity Hub reads with REST API queries and integrates API data into hub pages, history, rewards, and account transactions.

  • Adds list, detail, and historical Liquidity Hub API queries with React Query hooks.
  • Formats API hub, yield-group, reward, and transaction data for existing UI models.
  • Connects dashboard, header, form, loader, and transaction components to the fetched data.

Confidence Score: 5/5

The PR appears safe to merge based on the currently established evidence.

No blocking failure remains.

Important Files Changed

Filename Overview
apps/evm/src/clients/api/queries/getLiquidityHubs/index.ts Adds the REST query and formatting pipeline for the Liquidity Hub collection.
apps/evm/src/clients/api/queries/getLiquidityHub/index.ts Replaces the mock-backed single-hub lookup with an account-aware REST query.
apps/evm/src/clients/api/queries/getLiquidityHubHistory/index.ts Adds historical Liquidity Hub snapshot retrieval and normalization.
apps/evm/src/clients/api/queries/getAccountTransactionHistory/formatApiTransaction/formatToLiquidityHubTransaction/index.ts Adds migration debit and credit legs, but the prior migration-source concern cannot be conclusively reclassified from the available API semantics.
apps/evm/src/utilities/formatToLiquidityHub/index.ts Converts API Liquidity Hub records into the application’s display and transaction model.
apps/evm/src/containers/LiquidityHubForm/Form/index.tsx Connects form simulation and related-hub state to fetched Liquidity Hub data.

Reviews (2): Last reviewed commit: "feat: connect Liquidity Hub to API" | Re-trigger Greptile

@therealemjy

Copy link
Copy Markdown
Member Author

@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: 39418ecbba

ℹ️ 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".

Comment thread apps/evm/src/pages/LiquidityHub/LiquidityHubHistory/index.tsx
"higherThanWalletSpendingLimit": "Girdiğiniz miktar, bu varlık için harcama limitinizi aşıyor. Bu işleme devam etmek için lütfen mevcut harcama limitinizi iptal edin ve ardından yeni bir limit belirleyin.",
"smallerThanMinimumAmount": "{{minimumAmount}} değerinden daha azını taşıyamazsınız",
"smallerThanMinimumAmount": "TRANSLATION NEEDED",
"supplyCapReached": "Bu likidite merkezi için arz limiti {{supplyCap}} değerine ulaştı. Çekimler yapılana veya arz limiti artırılana kadar artık sağlayamazsınız."

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

TRANSLATION NEEDED

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Good catch

@cuzz-venus cuzz-venus left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All good for me, except the translation and conflicts

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.

2 participants