Implement `hasTargetUser()` helper function for remaining commands - Githubissues

CodecademyCommunity / codecademy-discord-bot

Custom moderation bot for the official Codecademy Community Discord server.

https://codecademycommunity.github.io/

MIT License

11 stars 1 forks source link

Implement `hasTargetUser()` helper function for remaining commands #214

Closed lyallstewart closed 2 years ago

lyallstewart commented 2 years ago

What issue is this solving?

Closes #213

Description

Updates the remaining commands to use the hasTargetUser helper function to reduce repetition and reuse the fun error messages.

Any helpful knowledge/context for the reviewer?

Is a re-seeding of the database necessary? ❌
Any new dependencies to install? ❌
Any special requirements to test? ❌
- (e.g., admin perms, alt account, etc.)

Please make sure you've attempted to meet the following coding standards

[X] Code has been tested and does not produce errors
[X] Code is readable and formatted
[X] There isn't any unnecessary commented-out code

lyallstewart commented 2 years ago

I've also tested all of the commands to make sure that I haven't done something silly and broken them - they all work as they should 😄

lyallstewart commented 2 years ago

Hi @aedwardg, Thanks for the review! Yeah my bad - that makes sense, I'll close this PR and open it out into some smaller ones 😃.