Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug fixes and removed useless stuff #1

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

malsvn
Copy link

@malsvn malsvn commented Dec 30, 2023

  • Fixed bot argument (not needed since we are using a selfbot package)
  • Removed useless imports from ghost.py
  • Removed the chunking of guilds at startup which may slow down the overall selfbot performance
  • Fixed commands not actually working due to them needing to be awaited

What issues the old version had (yours)

  • Commands were not added example shown here
  • Useless imports (may be slowing down performance)
  • Had bot argument from run() when it wasn't a known argument by discord.py-self

Edited:

  • Improved README by adding a fix for Improper Token Passed

- Fixed `bot` argument (not needed since we are using a selfbot package)
- Removed useless imports from `ghost.py`
- Removed the chunking of guilds at startup which may slow down the overall selfbot performance
- Fixed commands not actually working due to them needing to be awaited
- Added a potential fix tutorial for `discord.py-self`
@malsvn
Copy link
Author

malsvn commented Dec 30, 2023

Also fixed useless discord.py-self logging

@bennyscripts
Copy link
Owner

The "useless imports" are used for the script system, since the scripts are read and then ran through the main file all the imports need to be there. If you bring back the imports I'll consider merging.

@malsvn
Copy link
Author

malsvn commented Jan 2, 2024

oh okay my bad hold on

@malsvn
Copy link
Author

malsvn commented Jan 2, 2024

should have the imports now

@Euro-pol
Copy link

Euro-pol commented Jan 8, 2024

Thanks

@malsvn
Copy link
Author

malsvn commented Jan 11, 2024

Thanks

💯💯💯💯💯

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.

3 participants