Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent or warn spending ETH/MATIC when it brings trouble #2632

Open
norwnd opened this issue Dec 14, 2023 · 2 comments
Open

Prevent or warn spending ETH/MATIC when it brings trouble #2632

norwnd opened this issue Dec 14, 2023 · 2 comments

Comments

@norwnd
Copy link
Contributor

norwnd commented Dec 14, 2023

Currently user needs to have some non-0 MATIC amount in his wallet to trade on DEX against Polygon pairs (even if he wants to buy MATIC with his DCR/BTC/...).

However he can, User gets some MATIC to fund his very first actions with Polygon pairs. It's especially hard to do it in KYC-free way these days, but suppose he managed to do so. Then he must be careful with his MATIC balance to NOT let it drop below a certain point where he has to go through this complicated procedure of getting MATIC from external sources again.

It would be nice to have protective mechanisms for this in place to:

  • prevent(or warn) sending ERC20s in Polygon, if it results in "dangerously low" MATIC balance left
  • prevent(or warn) placing trades, if it results in "dangerously low" MATIC balance left
@norwnd norwnd changed the title Consider warning user when his ETH/MATIC balance gets low Prevent or warn spending ETH/MATIC when it brings trouble Dec 14, 2023
@xaur
Copy link
Contributor

xaur commented Feb 8, 2024

Is this specific to ETH and MATIC or all tradeable assets? e.g. can the user lose the ability to trade DCR pairs if the DCR balance drops below some level?

@norwnd
Copy link
Contributor Author

norwnd commented Feb 9, 2024

Is this specific to ETH and MATIC or all tradeable assets? e.g. can the user lose the ability to trade DCR pairs if the DCR balance drops below some level?

It's specifically for EVM-based chains DEX currently supports. Fundamentally, the same exact gotcha exists in most wallets I've used for gas-based chains (like Ethereum and Solana). So, it's not DEX-related thing per se, it's just a bit more prominent with DEX (and a significant UX improvement at low implementation cost).

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

No branches or pull requests

2 participants