-
Notifications
You must be signed in to change notification settings - Fork 356
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
Check for / reject zero amount tokens #76
Comments
As part of this, let's pull out the This already has a This would be a great help to build the higher-level checks for native coins in all contracts. I would also suggest 2 PRs here: one for the cw20 additions, one for the native coin changes (that I mentioned here) |
Yes, I was taking a look at the subkeys code, and was going to suggest exactly that. |
Correct. |
Add a check for considering zero amounts as missing.
For native coins, filter out zeros, and then ensure then remaining balance has length > 0. Only save the filtered list.
For cw20 coins, reject zero amounts directly. Add a check to also reject such sends in cw20-base.
The text was updated successfully, but these errors were encountered: