Skip to content

fix(wallet): point balance display transaction link to rewards page (#516) - #528

Open
lmduy2612 wants to merge 1 commit into
ChainLearnOfficial:mainfrom
lmduy2612:fix/balance-display-transactions-route-516
Open

lmduy2612 wants to merge 1 commit into
ChainLearnOfficial:mainfrom
lmduy2612:fix/balance-display-transactions-route-516

Conversation

@lmduy2612

Copy link
Copy Markdown

Summary

Closes #516.

Updates the "View Transactions" navigation link in BalanceDisplay component (src/components/wallet/balance-display.tsx) to point to the existing /rewards page instead of the non-existent /transactions route, eliminating 404 navigation errors.

Changes Made

  • Updated href in src/components/wallet/balance-display.tsx from /transactions to /rewards.
  • Updated unit test in src/tests/wallet/balance-display.test.tsx to verify navigation link targets /rewards.
  • Ran unit tests (5/5 passed) and verified ESLint passes cleanly.

Acceptance Criteria

  • Point the link to the rewards page (/rewards) which already shows reward and transaction history

@netlify

netlify Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

❌ Deploy Preview for chainlearn failed.

Name Link
🔨 Latest commit b278bd8
🔍 Latest deploy log https://app.netlify.com/projects/chainlearn/deploys/6abf7177806e56000704f671

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.

Balance display links to non-existent /transactions route

1 participant