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

Upgrade Dependencies for Discord.py 2.1.0 #134

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

JustinR17
Copy link
Collaborator

Discord.py v1 will be unusable in 2023 due to discord api v7 being decommissioned (discord.py migration -- discord docs change log decommissioning notice).

Currently the clot uses python 3.7, but this will need to be upgraded to at least 3.8 to support new features that discord.py uses. This means that Django and a few other python libs need to be upgraded as a consequence. README has been updated to reflect these changes in addition to fixing warnings raised by the new Django version.

Currently testing changes with the new Django & discord.py versions

@JustinR17 JustinR17 changed the title [DRAFT] Upgrade Dependencies for Discord.py 2.1.0 Upgrade Dependencies for Discord.py 2.1.0 Dec 27, 2022
@JustinR17 JustinR17 added dependencies Pull requests that update a dependency file and removed DO NOT MERGE labels Dec 27, 2022
@JustinR17
Copy link
Collaborator Author

Tested running the above using the latest python 3.10 version (3.10.9). Upgrading pip to 22.3.1 seems to fix any issues with dependencies not installing.

PR also updates code to fix warnings (mainly using is when == of != should be used. Running python manage.py runserver, python manage.py bot, python manage.py engine, and python manage.py server all appear to work.

@JustinR17 JustinR17 force-pushed the jr-upgrade-discord-py-2.1 branch 4 times, most recently from 3a6760a to a3a793b Compare December 28, 2022 01:42
@JustinR17
Copy link
Collaborator Author

Pushed a new commit to change the hardcoded IDs for clan league tournaments for clan league 16 (id=777)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant