-
-
Notifications
You must be signed in to change notification settings - Fork 364
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
fix: Auto teleport - user should have at least 1.5 DOT #9953
Conversation
✅ Deploy Preview for koda-canary ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
nice finding, it was this then that was causing the bug, ty edit: exez found it first lol |
If it works let's merge it. The code you have commented out is unreadable for me, sorry |
fair enough, can remove it if you want the entire auto-teleport needs some refactoring to be done #9695 |
Quality Gate passedIssues Measures |
Code Climate has analyzed commit ef0c07a and detected 0 issues on this pull request. View more on Code Climate. |
@vikiival can we get approval? |
pay 15 usd |
😍 Perfect, I’ve sent the payout 🪅 Let’s grab another issue and get rewarded! |
PR Type
Context
based on
ED is already 1.5 DOT, disabled the feature that allowed the autotelport of all balance in the case of slashing,
now you can only autotelport any sum above of 1.5
just to make sure tested all the remaining autotelport logic , as far I can see it's working as expected. @prury please test
Needs QA check
Did your issue had any of the "$" label on it?
Screenshot 📸