search

skiqqy / C45DiscordBot

Proprietary™ (Thats a joke btw) C45 discord bot.
MIT License
2 stars 1 forks source link

delete X messages #43

Closed lakshyashastri closed 3 years ago

lakshyashastri commented 3 years ago

Should solve #21

skiqqy commented 3 years ago

I shall take a look asap (just have a some assignments due today), thanks for the solution @LakshyaShastri!

skiqqy commented 3 years ago

Hi there apologies for only taking a look now, looks good. I would just prefer for it to be included in the commands section. That is refactor the added code into bot_command.py, hence it will be invoked when we call /del 2 for example. Once that is done ill happily merge :)

Enjoy the rest of your evening!

lakshyashastri commented 3 years ago

The "commands section" doesn't have bot commands, from what I gather it's for running shell commands and I can't just copy-paste this code there since the function isn't asynchronous either

skiqqy commented 3 years ago

Oh i see the confusion, no the commands section is for invoking commands through discord, using the command prefix. At line 134 in __main__.py, as that is where the commands get parsed.

Thanks!

lakshyashastri commented 3 years ago

@Skippy404 Have a look now!

skiqqy commented 3 years ago

Yo @LakshyaShastri! I took a look, and there is just a small bug (line 130, __main__.py),

provided in the screenshot below is the fix, apply it and ill merge \:D image

Enjoy the rest of your evening, and thanks for jumping in on this issue!

lakshyashastri commented 3 years ago

@Skippy404 No problem, did that!

skiqqy commented 3 years ago

Thanls @LakshyaShastri \:D