Skip to content

remove old amm page components#1306

Merged
kuan121 merged 3 commits intomainfrom
remove-old-amm-object-page
Apr 10, 2026
Merged

remove old amm page components#1306
kuan121 merged 3 commits intomainfrom
remove-old-amm-object-page

Conversation

@kuan121
Copy link
Copy Markdown
Contributor

@kuan121 kuan121 commented Apr 1, 2026

High Level Overview of Change

Remove old AMM page components. The revamped AMM Pool/Object page will be added via the PR #1305. This PR will be merged after #1305 is merged.

Context of Change

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactor (non-breaking change that only restructures code)
  • Tests (You added tests for code that already exists, or your new feature included in this PR)
  • Documentation Updates
  • Translation Updates
  • Release

Codebase Modernization

  • Updated files to React Hooks
  • Updated files to TypeScript

Before / After

Test Plan

ckeshava
ckeshava previously approved these changes Apr 1, 2026
"last_affecting_transaction": "နောက်ဆုံး သက်ရောက်မှုရှိသော ငွေလွှဲမှု",
"Version": "ဗားရှင်း",
"increased_by": "တိုးလာသည်",
"lp_token_balance": "LP Token လက်ကျန်",
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.

If these tokens are used in the new AMM page, it will be useful to preserve the international translations.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good point. We can reuse lp_token_balance. I will keep it then.

@kuan121 kuan121 force-pushed the remove-old-amm-object-page branch from d2823b3 to 228b732 Compare April 10, 2026 19:41
@kuan121 kuan121 merged commit 4f0dab2 into main Apr 10, 2026
6 checks passed
@kuan121 kuan121 mentioned this pull request Apr 14, 2026
8 tasks
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.

3 participants