Portfolio: hide test-gem Open Positions and P&L by default (#674) #1177

Merged
PlasticDigits merged 1 commit from issue-674-portfolio-hide-test-gems into main 2026-08-27 04:44:31 +00:00
PlasticDigits commented 2026-08-27 00:33:34 +00:00 (Migrated from gitlab.com)

Closes #674

Summary

  • /portfolio Open Positions, header realized P&L (visible rows), and Recent activity hide soft-launch test gems by default (EMBER, CORAL, JADE, TOPAZ, ONYX, RUBY, PEARL, QUARTZ, plus LocalTerra extras).
  • Reuses the existing #534 / #562 classifier (isTestPair / isGemTokenId) — no second gem list.
  • Show test pairs toggle (default off) reveals gem rows under a Test pairs divider so QA can still inspect them.
  • /trader, LP overview, and open limits stay unfiltered. Indexer JSON is unchanged. total_volume_usd stays the indexer-wide wallet figure.

Test plan

  • make verify-issue-674
  • Connected /portfolio with a gem position (e.g. EMBER/CORAL): row hidden by default; header P&L excludes it
  • Toggle Show test pairs: gem row appears after a Test pairs divider; P&L includes it
  • Economic pairs (CL8Y, UST1, USTR, cLUNC, cUSTC, vFDUSD) still list
  • /trader/:addr still shows gem positions without a hide toggle
  • Open limits and LP overview still show gem pairs / balances
  • Toggle is absent when the wallet has no gem performance
  • Playwright e2e/portfolio.spec.ts when LocalTerra is up (disconnected visit does not show the toggle)
Closes #674 ## Summary - `/portfolio` Open Positions, header realized P&L (visible rows), and Recent activity hide soft-launch test gems by default (EMBER, CORAL, JADE, TOPAZ, ONYX, RUBY, PEARL, QUARTZ, plus LocalTerra extras). - Reuses the existing `#534` / `#562` classifier (`isTestPair` / `isGemTokenId`) — no second gem list. - **Show test pairs** toggle (default off) reveals gem rows under a **Test pairs** divider so QA can still inspect them. - `/trader`, LP overview, and open limits stay unfiltered. Indexer JSON is unchanged. `total_volume_usd` stays the indexer-wide wallet figure. ## Test plan - [ ] `make verify-issue-674` - [ ] Connected `/portfolio` with a gem position (e.g. EMBER/CORAL): row hidden by default; header P&L excludes it - [ ] Toggle **Show test pairs**: gem row appears after a **Test pairs** divider; P&L includes it - [ ] Economic pairs (CL8Y, UST1, USTR, cLUNC, cUSTC, vFDUSD) still list - [ ] `/trader/:addr` still shows gem positions without a hide toggle - [ ] Open limits and LP overview still show gem pairs / balances - [ ] Toggle is absent when the wallet has no gem performance - [ ] Playwright `e2e/portfolio.spec.ts` when LocalTerra is up (disconnected visit does not show the toggle)
PlasticDigits commented 2026-08-27 00:33:43 +00:00 (Migrated from gitlab.com)

mentioned in issue #674

mentioned in issue #674
PlasticDigits (Migrated from gitlab.com) merged commit e4bab0fb4e into main 2026-08-27 04:44:31 +00:00
PlasticDigits commented 2026-08-27 04:44:35 +00:00 (Migrated from gitlab.com)

mentioned in commit e4bab0fb4e

mentioned in commit e4bab0fb4e49a4656e5764c1b4fbf8943612c77c
PlasticDigits commented 2026-08-27 05:29:52 +00:00 (Migrated from gitlab.com)

mentioned in issue #686

mentioned in issue #686
PlasticDigits commented 2026-08-27 07:21:02 +00:00 (Migrated from gitlab.com)

mentioned in commit 3ff8d00b2d

mentioned in commit 3ff8d00b2dd4390e5b6784b8a325f3364f646f17
PlasticDigits commented 2026-08-29 04:53:39 +00:00 (Migrated from gitlab.com)

mentioned in issue #678

mentioned in issue #678
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!1177
No description provided.