Skip to content

Force max unstake on Kiln ETH positions#5453

Merged
peachbits merged 1 commit intodevelopfrom
matthew/kiln-eth-maxunstake
Feb 18, 2025
Merged

Force max unstake on Kiln ETH positions#5453
peachbits merged 1 commit intodevelopfrom
matthew/kiln-eth-maxunstake

Conversation

@peachbits
Copy link
Contributor

@peachbits peachbits commented Feb 13, 2025

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

  • Yes
  • No

Dependencies

none

Requirements

If you have made any visual changes to the GUI. Make sure you have:

  • Tested on iOS device
  • Tested on Android device
  • Tested on small-screen device (iPod Touch)
  • Tested on large-screen device (tablet)

@peachbits peachbits force-pushed the matthew/kiln-eth-maxunstake branch from dae74f9 to cb82451 Compare February 13, 2025 01:34
@peachbits peachbits force-pushed the matthew/kiln-eth-maxunstake branch from cb82451 to 8e70201 Compare February 17, 2025 23:33
@peachbits peachbits enabled auto-merge February 17, 2025 23:34
@peachbits peachbits force-pushed the matthew/kiln-eth-maxunstake branch from 8e70201 to f1f98e0 Compare February 17, 2025 23:50
This is a hack to prevent users from being able to enter a greater amount of LP tokens than they actually have. Since the currency plugins and GUI's stake plugins all still rely on currency code, the actual balance of LP tokens isn't readily available and would recover some hacking to override the balance used in the flip input modal. We're not passing the correct tokenId in anyway so introducing hack in the stake scenes and flip input could just cause more issues.
@peachbits peachbits force-pushed the matthew/kiln-eth-maxunstake branch from f1f98e0 to 9c6a1c3 Compare February 18, 2025 00:05
@peachbits peachbits merged commit d8f9ec9 into develop Feb 18, 2025
2 checks passed
@peachbits peachbits deleted the matthew/kiln-eth-maxunstake branch February 18, 2025 00:25
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.

2 participants