Where we were supposed to be filtering out the command initiator it was doing a string vs int comparison 🤦
Went ahead and also fired off a wishlist purge when the command is run as well for good measure (simply clears out a users wishlist of any badges that they currently already possess, so the later checks should now only provide redundancy).
We should also run the purge when a user receives a new scrap badge so adding that as well.
See #374
Where we were supposed to be filtering out the command initiator it was doing a string vs int comparison 🤦
Went ahead and also fired off a wishlist purge when the command is run as well for good measure (simply clears out a users wishlist of any badges that they currently already possess, so the later checks should now only provide redundancy).
We should also run the purge when a user receives a new scrap badge so adding that as well.