Skip to content

Conversation

pennstatephil
Copy link

I think it'd be helpful to know which words and channels are being ignored, so you can add/remove from those lists as needed. I'm new to contributing to discord bots, so I'm not sure how to test the new commands; but I believe these will work.

Copy link
Owner

@ducktrshessami ducktrshessami left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I apologize that this may sound/be a little obtuse.

As mentioned in the README, this is a port of someone else's Twitch/IRC bot. I've done this a couple times mainly for friends, but generally when I do this, I try not to overstep how the original version of the bot behaves/feels unless necessary. In the case of the word blacklist commands, I could consider those valuable to have since it'd be impossible to achieve that sort of customization without them (and this is important for the reasons I described on your feature request). However with ignore list commands, I find these less necessary; one can still customize the experience without the ability to view current customization in summary. Therefore, I'd rather respect the original buttsbot's behavior of having fewer features like this. It didn't really have a means of checking whether the bot is in your channel, whether you're ignored, or what any of the buttification configuration was set to.

There are further exceptions to this such as how with my version, you can explicitly check buttification configuration using the same commands to set configuration and how the count command is missing. For the former, this is due to my ability to naturally merge this with already existing commands instead of needing to create ones that didn't previously exist. For the latter, this is due to my personally being against the feature as a whole. I consider channel counting commands to be entirely extraneous on any bot, providing no real value to anyone.

All of this isn't to say that I don't understand the usefulness of having these list commands. It's mostly just artistic preference, I suppose.

If you truly wish to have this, you always have the option of hosting your own fork.

@ducktrshessami
Copy link
Owner

ducktrshessami commented Dec 27, 2024

Some side notes:

  • People generally test features by running a separate instance of the bot within a controlled environment. Discord has not provided any sort of command testing tools at this time.
  • Similarly for the other feature you mentioned previously, I'd rather not create an extra command for bulk-ignoring words. However, if you wish to add this onto the existing command (e.g. with comma-separation), I would accept a PR.

@pennstatephil
Copy link
Author

Appreciate the feedback! Totally understand, and I may look into what it would take to host it myself just for these fun add on ideas. Appreciate the addition of the word blacklist, regardless. Happy holidays!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants