Skip to content

Makes it easier to install and run, less problems. Also utilizes discord.py's command module instead of on_message#35

Open
patttterson wants to merge 11 commits intoNPCat:mainfrom
patttterson:main
Open

Makes it easier to install and run, less problems. Also utilizes discord.py's command module instead of on_message#35
patttterson wants to merge 11 commits intoNPCat:mainfrom
patttterson:main

Conversation

@patttterson
Copy link

Changes

Updated bad apple discord player.py to use discord.py's built-in command module instead of on_message. (See this.)

Deleted Info to run.txt and replaced it with README.md, which provides better instructions

Created requirements.txt, which makes it easier to install the required packages

Updated video to frames.py by making it put frames into a frames folder instead of spitting it out into the main directory

Side note: this PR took like an hour of work, but it will probably save the two people that want to try this for themselves a heck more time.

Updated it to use discord.py v2.1.0, also changed the command to use the commands module instead of on_message
Remove this file in favor of the README.md file instead
Make it easier to install the necessary packages
Added more instructions that I missed
Puts the frames into the frames directory, instead of just spitting them out
Creates frames directory to store the frames in
TYPO!!! YEAHHH
@patttterson
Copy link
Author

Also, @NPCat , if you see this, consider taking a look at the other PRs. They have other important things that I missed here.

@patttterson patttterson marked this pull request as draft December 2, 2022 22:40
@patttterson patttterson marked this pull request as ready for review December 2, 2022 22:42
@limelambda
Copy link

@bot.event
async def on_ready():
print(f'I have logged in as {client} (ID: {client.id})')

Dosen't seem to work?

@patttterson
Copy link
Author

Sorry, just fixed this.

honestly this repository is probably completely dead so i dont really care anymore

it was a stupid mistake lmaoo

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

Comments