Skip to content

fix: prevent user from sending 0 NFTs#24420

Merged
BZahory merged 10 commits intomb843/updatesfrom
zero-asset-fix
May 8, 2024
Merged

fix: prevent user from sending 0 NFTs#24420
BZahory merged 10 commits intomb843/updatesfrom
zero-asset-fix

Conversation

@BZahory
Copy link
Copy Markdown
Contributor

@BZahory BZahory commented May 8, 2024

No description provided.

ejwessel and others added 10 commits May 7, 2024 13:41
Signed-off-by: Ethan Wessel <ethan.wessel@consensys.net>
Signed-off-by: Ethan Wessel <ethan.wessel@consensys.net>
Signed-off-by: Ethan Wessel <ethan.wessel@consensys.net>
Signed-off-by: Ethan Wessel <ethan.wessel@consensys.net>
Signed-off-by: Ethan Wessel <ethan.wessel@consensys.net>
Signed-off-by: Ethan Wessel <ethan.wessel@consensys.net>
keeps everything in one line
we can't tell if it's actually zero or empty
@BZahory BZahory requested a review from a team as a code owner May 8, 2024 00:07
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 8, 2024

CLA Signature Action:

Thank you for your submission, we really appreciate it. We ask that you all read and sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just by adding a comment to this pull request with this exact sentence:

I have read the CLA Document and I hereby sign the CLA

By commenting with the above message you are agreeing to the terms of the CLA. Your account will be recorded as agreeing to our CLA so you don't need to sign it again for future contributions to this repository.

1 out of 2 committers have signed the CLA.
@BZahory
@ejwessel

@BZahory BZahory merged this pull request into mb843/updates May 8, 2024
@BZahory BZahory deleted the zero-asset-fix branch May 8, 2024 00:07
@github-actions github-actions bot locked and limited conversation to collaborators May 8, 2024
@metamaskbot metamaskbot added the INVALID-PR-TEMPLATE PR's body doesn't match template label May 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

INVALID-PR-TEMPLATE PR's body doesn't match template

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants