Skip to content

feat: promote the liquidity hub in the top bar and landing page - #5797

Open
cuzz-venus wants to merge 2 commits into
feat/liquidity-hub-apifrom
feat/liquidity-hub-topbar
Open

feat: promote the liquidity hub in the top bar and landing page#5797
cuzz-venus wants to merge 2 commits into
feat/liquidity-hub-apifrom
feat/liquidity-hub-topbar

Conversation

@cuzz-venus

Copy link
Copy Markdown
Contributor

Jira ticket(s)

VPD-1912

Changes

  • Moved the Dashboard entry from the left of the top bar to the right, just ahead of the reward claim button, and renamed the Markets menu to Borrow.
  • Pointed the landing page call to action at the liquidity hub as "Earn now", keeping the previous "Start now" link to the core markets on chains where the hub is not deployed.

Move the dashboard entry to the right of the top bar, ahead of the reward
claim button, rename the markets menu to Borrow, and point the landing
page call to action at the liquidity hub where it is available.
@vercel

vercel Bot commented Aug 27, 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 27, 2026 3:47pm
dapp-testnet Ready Ready Preview Aug 27, 2026 3:47pm
venus.io Ready Ready Preview Aug 27, 2026 3:47pm

Request Review

@changeset-bot

changeset-bot Bot commented Aug 27, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 2566310

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@greptile-apps

greptile-apps Bot commented Aug 27, 2026

Copy link
Copy Markdown

Greptile Summary

The PR promotes Liquidity Hub through the landing-page CTA, moves Dashboard into the top-bar action group, and renames Markets to Borrow.

  • Routes the landing CTA to Liquidity Hub when the feature is enabled.
  • Preserves the core-markets CTA on chains without Liquidity Hub.
  • Supplies localized labels for the new CTA and Borrow menu across all supported locales.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
apps/evm/src/pages/Landing/Hero/index.tsx Selects the Liquidity Hub CTA route and label when enabled while preserving the chain-aware markets fallback.
apps/evm/src/containers/Layout/NavBar/index.tsx Moves the desktop Dashboard link into the action group while retaining it in mobile navigation.
apps/evm/src/containers/Layout/NavBar/useMenuItems/index.tsx Extracts the Dashboard menu item and renames the Markets navigation group to Borrow.
apps/evm/src/libs/translations/translations/ja.json Replaces the previously reported placeholders with Japanese translations for the CTA and Borrow label.
apps/evm/src/libs/translations/translations/th.json Replaces the previously reported placeholders with Thai translations for the CTA and Borrow label.
apps/evm/src/libs/translations/translations/tr.json Replaces the previously reported placeholders with Turkish translations for the CTA and Borrow label.
apps/evm/src/libs/translations/translations/vi.json Replaces the previously reported placeholders with Vietnamese translations for the CTA and Borrow label.
apps/evm/src/libs/translations/translations/zh-Hans.json Replaces the previously reported placeholders with Simplified Chinese translations for the CTA and Borrow label.
apps/evm/src/libs/translations/translations/zh-Hant.json Replaces the previously reported placeholders with Traditional Chinese translations for the CTA and Borrow label.

Reviews (2): Last reviewed commit: "feat: add translation" | Re-trigger Greptile

Comment thread apps/evm/src/libs/translations/translations/ja.json Outdated
@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Coverage Report for ./apps/evm

Status Category Percentage Covered / Total
🔵 Lines 80.88% 52796 / 65274
🔵 Statements 80.88% 52796 / 65274
🔵 Functions 74.52% 1612 / 2163
🔵 Branches 85.94% 7764 / 9034
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
apps/evm/src/containers/Layout/testIds.ts 100% 100% 100% 100%
apps/evm/src/containers/Layout/NavBar/index.tsx 100% 95% 100% 100%
apps/evm/src/containers/Layout/NavBar/useMenuItems/index.tsx 96.55% 75% 100% 96.55% 81-85
apps/evm/src/pages/Landing/Hero/index.tsx 100% 71.42% 100% 100%
Generated in workflow #14183 for commit 2566310 by the Vitest Coverage Report Action

@cuzz-venus

Copy link
Copy Markdown
Contributor Author

@greptile review again

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