Skip to content

Fix pool LP lending withdraw and success screen UX#477

Merged
temptemp3 merged 2 commits into
betafrom
fix/pools-lp-withdraw-and-success-ui
Jun 4, 2026
Merged

Fix pool LP lending withdraw and success screen UX#477
temptemp3 merged 2 commits into
betafrom
fix/pools-lp-withdraw-and-success-ui

Conversation

@temptemp3
Copy link
Copy Markdown
Contributor

Summary

  • Align pool-page LP withdraw with Portfolio: HF-safe max, pool collateral rows, and borrow-aware poolHasNoBorrows instead of a hardcoded value
  • Defer pool list refresh until after the withdraw success modal renders so congrats are not wiped by query invalidation
  • Improve LP success screens with pair icons, LP labels, and a signed-amount snapshot; enable supply/withdraw when Tinyman snapshot is unavailable

Test plan

  • On Pools page, supply LP to platform and confirm success screen shows pair icons, amount, and label (e.g. WAD / ALGO LP)
  • Withdraw partial and max LP; confirm success screen appears and stays visible before pool cards refresh
  • With an open WAD borrow against LP collateral, confirm withdraw respects HF-safe max and does not allow unsafe full withdraw
  • Confirm Supply/Withdraw buttons work when Tinyman pool snapshot fails to load but on-chain supplied balance exists
  • Verify Portfolio withdraw success flow still works (shared WithdrawModal changes)

Made with Cursor

Wire pool-page withdraw like Portfolio with HF-safe max, collateral rows,
and real borrow state. Defer pool refresh until after the success modal
renders so LP congrats are not wiped by query invalidation. Show pair
icons and LP labels on success screens; enable supply/withdraw when
Tinyman snapshot is unavailable.

Co-authored-by: Cursor <cursoragent@cursor.com>
@temptemp3 temptemp3 changed the base branch from next to beta June 4, 2026 00:18
@temptemp3 temptemp3 merged commit 2b2add1 into beta Jun 4, 2026
temptemp3 added a commit that referenced this pull request Jun 4, 2026
* Hide Pool C/E LP markets from portfolio page (#476)

Apply the same Markets table exclusion rules to portfolio position lists
and market snapshots so Tinyman LP collateral on pools C and E no longer
appear in supplied/borrowed assets, while WAD borrow on those pools
stays visible.

Co-authored-by: Cursor <cursoragent@cursor.com>

* Fix pool LP lending withdraw and success screen UX (#477)

Wire pool-page withdraw like Portfolio with HF-safe max, collateral rows,
and real borrow state. Defer pool refresh until after the success modal
renders so LP congrats are not wiped by query invalidation. Show pair
icons and LP labels on success screens; enable supply/withdraw when
Tinyman snapshot is unavailable.

Co-authored-by: Cursor <cursoragent@cursor.com>

---------

Co-authored-by: Cursor <cursoragent@cursor.com>
@temptemp3 temptemp3 deleted the fix/pools-lp-withdraw-and-success-ui branch June 4, 2026 00:33
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