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

Update discord py #40

Open
wants to merge 16 commits into
base: dev
Choose a base branch
from
Open

Update discord py #40

wants to merge 16 commits into from

Conversation

getsbuffer
Copy link

@getsbuffer getsbuffer commented Mar 2, 2025

Update discord-py to 2.5.0 in Pipfile, requirements, requirements-dev. Change webhook implementation in announcements.

Resolves #30 #32 #33 #34 #35


## Misc

The following variables, located in the main settings file[^1], map to miscellaneous PCS specific settings. A link to relevant documentation is provided for each variable.
The following variables, located in the main settings file[^2], map to miscellaneous PCS specific settings. A link to relevant documentation is provided for each variable.
Copy link
Member

Choose a reason for hiding this comment

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

The footnote has been updated to the incorrect settings file. The footnote should be [^1].

@@ -24,7 +24,7 @@ REQUIREMENTS | requirements.txt | Path to the manifest file to use during image

## Run Image

The `Dockerfile` specifies an image capable of running the Django web app, Celery, or a helper Discord bot. The parameters passed to the image's start script `start.sh` determine the container's behavior.
The `Dockerfile` specifies an image capable of running the Django web app with Celery. The parameters passed to the image's start script `start.sh` determine the container's behavior.
Copy link
Member

Choose a reason for hiding this comment

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

Update to: ...specifies an image capable of running the Django web app, Celery, or Flower.

Copy link
Member

@mmcinnestaylor mmcinnestaylor left a comment

Choose a reason for hiding this comment

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

Great work. I added a few comments to files which need tweaks before we merge the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
3 participants