fix(frontend): /pool Manage four peer actions (#660) #1175

Merged
PlasticDigits merged 3 commits from feat/660-pool-manage-ia into main 2026-08-26 09:19:28 +00:00
PlasticDigits commented 2026-08-26 07:41:22 +00:00 (Migrated from gitlab.com)

Summary

  • /pool no longer leads with page-level Add/Withdraw zap cards. Pair Manage exposes four peer actions: Provide Liquidity, Withdraw Liquidity, Zap Add, Zap Withdraw.
  • Zap is pinned to the expanded factory row (no second pair/LP picker). Empty-pool zap is disabled with Empty pool. Use Provide Liquidity. Two-sided provide stays the first-deposit path.
  • Docs, glossary, how-to, and skills/AGENTS_FRONTEND_POOL_MANAGE_IA.md (M660-1–M660-8) replace “two-sided is Advanced.” make verify-issue-660 covers units, greps, Playwright smoke (5 workers), and LocalTerra provide + zap-add tx (1 worker) on Vite :3173.

Closes #660

Test plan

  • PLAYWRIGHT_WEB_PORT=3173 make verify-issue-660 (units + greps + T1–T6/T13 smoke @ 5 workers + T19 provide + P4 zap-add tx @ 1 worker)
  • make verify-issue-531 / 533 / 547 / 559 / 653 (e2e skipped on 531/533/547/559 in this run; smoke already green for 531/547/660/533 UI)
  • pool.spec.ts + pool-lp-howto-531.spec.ts + pool-table-547.spec.ts smoke (25/25)
  • Reviewer: /pool default paint has no zap cards; Manage shows four tabs without Advanced
  • Reviewer: empty pool zap disabled, copy points at Provide Liquidity
  • Reviewer: 375px tabs wrap and stay reachable
  • Reviewer: pre-sign action matches the selected tab (Zap Add ≠ Provide Liquidity)
## Summary - `/pool` no longer leads with page-level Add/Withdraw zap cards. Pair **Manage** exposes four peer actions: **Provide Liquidity**, **Withdraw Liquidity**, **Zap Add**, **Zap Withdraw**. - Zap is pinned to the expanded factory row (no second pair/LP picker). Empty-pool zap is disabled with `Empty pool. Use Provide Liquidity.` Two-sided provide stays the first-deposit path. - Docs, glossary, how-to, and `skills/AGENTS_FRONTEND_POOL_MANAGE_IA.md` (**M660-1–M660-8**) replace “two-sided is Advanced.” `make verify-issue-660` covers units, greps, Playwright smoke (5 workers), and LocalTerra provide + zap-add tx (1 worker) on Vite `:3173`. Closes #660 ## Test plan - [x] `PLAYWRIGHT_WEB_PORT=3173 make verify-issue-660` (units + greps + T1–T6/T13 smoke @ 5 workers + T19 provide + P4 zap-add tx @ 1 worker) - [x] `make verify-issue-531` / `533` / `547` / `559` / `653` (e2e skipped on 531/533/547/559 in this run; smoke already green for 531/547/660/533 UI) - [x] `pool.spec.ts` + `pool-lp-howto-531.spec.ts` + `pool-table-547.spec.ts` smoke (25/25) - [ ] Reviewer: `/pool` default paint has no zap cards; Manage shows four tabs without Advanced - [ ] Reviewer: empty pool zap disabled, copy points at Provide Liquidity - [ ] Reviewer: 375px tabs wrap and stay reachable - [ ] Reviewer: pre-sign action matches the selected tab (Zap Add ≠ Provide Liquidity)
PlasticDigits commented 2026-08-26 07:50:56 +00:00 (Migrated from gitlab.com)

added 1 commit

  • 4608b451 - fix(frontend): run #660 pool-tx from Manage tabs on CORS-safe Vite

Compare with previous version

added 1 commit <ul><li>4608b451 - fix(frontend): run #660 pool-tx from Manage tabs on CORS-safe Vite</li></ul> [Compare with previous version](/PlasticDigits/cl8y-dex-terraclassic/-/merge_requests/457/diffs?diff_id=1984373573&start_sha=9c5c55e947ec62737acf9b47b3bbf7a0aeacc2dd)
PlasticDigits commented 2026-08-26 07:51:20 +00:00 (Migrated from gitlab.com)

changed the description

changed the description
PlasticDigits commented 2026-08-26 07:51:22 +00:00 (Migrated from gitlab.com)

mentioned in issue #660

mentioned in issue #660
PlasticDigits commented 2026-08-26 09:19:18 +00:00 (Migrated from gitlab.com)

mentioned in commit 7c5e693383

mentioned in commit 7c5e693383657e00d9977cb91781890ec312a930
PlasticDigits commented 2026-08-26 09:19:19 +00:00 (Migrated from gitlab.com)

added 49 commits

  • 4608b451...b64331ff - 48 commits from branch main
  • 7c5e6933 - Merge origin/main into branch for !1175

Compare with previous version

added 49 commits <ul><li>4608b451...b64331ff - 48 commits from branch <code>main</code></li><li>7c5e6933 - Merge origin/main into branch for !1175</li></ul> [Compare with previous version](/PlasticDigits/cl8y-dex-terraclassic/-/merge_requests/457/diffs?diff_id=1984604238&start_sha=4608b451e533e5c88a6f47d7f2ee7d718e1c9acf)
PlasticDigits commented 2026-08-26 09:19:28 +00:00 (Migrated from gitlab.com)

mentioned in commit 29ed0bdad3

mentioned in commit 29ed0bdad36cf9cb8120b54d658cc0c511600f27
PlasticDigits (Migrated from gitlab.com) merged commit 29ed0bdad3 into main 2026-08-26 09:19:28 +00:00
PlasticDigits commented 2026-08-26 09:30:56 +00:00 (Migrated from gitlab.com)

mentioned in issue #661

mentioned in issue #661
PlasticDigits commented 2026-08-26 09:31:37 +00:00 (Migrated from gitlab.com)

mentioned in issue #673

mentioned in issue #673
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
code/cl8y-dex-terraclassic!1175
No description provided.