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 modules, dockerfile and container. fix and change things that … #40

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tommywienert
Copy link

@tommywienert tommywienert commented Oct 14, 2024

…are handled differently now (mainly discords 'intents' system)

Hey there @jarulsamy, i found your project and realised it wasn't working anymore(as mentioned in #39 ), but it inspired me to do a nodejs Plex-Discord-Bot.
As a thank you, i wanted to fix your bot since you don't have time currently. So here it is. i updated everything to a newer state (i think discord does not allow the old stuff to work anymore ... you need those intents), if you have some time please have a look if this is okay for you.
in detail:
python modules are updated, i tested this on python3.12 but i left in a little check because of asyncio/async_timeout
Dockerfile now uses the python3.12-slim instead of python3.7-slim
and a slightly modified startup sequence.

greetings
Tommy

P.S.: To the users: if you are , like me, not a fan of docker you can also use this locally, you need (inside the Plex-Bot directory):
python3 -m venv env
source env/bin/activate
(env) pip install --upgrade pip setuptools wheel
(env) pip install -r requirements.txt
(env) python -m PlexBot

if you want out ... simply type
(env) deactivate

on linux you can also make a systemd service for it, the most important bit is
ExecStart=/path/to/env/bin/python -m PlexBot

…are handled differently now (mainly discords 'intents' system)
@tommywienert
Copy link
Author

Hey @jarulsamy , are you still with us? 🤔

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.

1 participant