chia-blockchain icon indicating copy to clipboard operation
chia-blockchain copied to clipboard

Xch spam

Open danieljperry opened this issue 2 years ago • 2 comments

Add new spam filter and tests, and make mergeable with main branch. This is to replace PR #11894

danieljperry avatar Aug 10 '22 08:08 danieljperry

This pull request introduces 1 alert when merging dd73b218f96df5c9ef80e1c861b929bd8c4b3beb into da69ae6ad1890147318f5b3b7c2dcdc8ca5f7b11 - view on LGTM.com

new alerts:

  • 1 for Redundant comparison

lgtm-com[bot] avatar Aug 10 '22 09:08 lgtm-com[bot]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Aug 10 '22 11:08 github-actions[bot]

Two more things @danieljperry.

  1. Can you fix the merge conflicts?
  2. How about adding a test where the user has > 200 dust coins, and then receives an NFT. I want to ensure that NFTs are never filtered (since they use 1 or few mojos).

mariano54 avatar Aug 23 '22 14:08 mariano54

Conflicts have been resolved. A maintainer will review the pull request shortly.

github-actions[bot] avatar Aug 25 '22 01:08 github-actions[bot]

This pull request introduces 1 alert when merging 91dc732e20ce29aeb9be304aba12c03cf418118c into b3b56a64d149ad521b3094ed9326c6aa2ae02fe4 - view on LGTM.com

new alerts:

  • 1 for Redundant comparison

lgtm-com[bot] avatar Aug 25 '22 01:08 lgtm-com[bot]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Aug 29 '22 23:08 github-actions[bot]

Conflicts have been resolved. A maintainer will review the pull request shortly.

github-actions[bot] avatar Aug 30 '22 10:08 github-actions[bot]