search

luigi311 / JellyPlex-Watched

Sync watched between jellyfin and plex locally
GNU General Public License v3.0
394 stars 22 forks source link

Draft: Plex refactor #72

Closed Nicba1010 closed 8 months ago

Nicba1010 commented 1 year ago

I'm refactoring plex.py a little bit to clean up extra code and make some processes more Pythonic in some places. This bumps the Python version to 3.9, I hope that isn't a problem. Most distros should have 3.9 by now.

luigi311 commented 1 year ago

yeah that should be fine since 3.9 is 3 years old now so it should be everywhere and those that dont have it for some reason can just run the container

luigi311 commented 1 year ago

Hello, just following up to see if you are still working on this or if this was completed but left as draft by accident

Nicba1010 commented 1 year ago

Hi, I'm still working on it, but work and life got in the way for a little bit.

luigi311 commented 1 year ago

No worries, theres nothing going on in the plex side of things so we shouldnt have to worry about any rebasing if it takes you a while.

luigi311 commented 9 months ago

Hello, I was wondering if it was alright if i merged this in as is since it looks like this is all great and seems to be working as is so i can mess around with the plex side afterwards to see if i can figure out how to make it less heavy. If needed you can then make a new PR with more improvements in the future when you have time.

luigi311 commented 8 months ago

Merged in #133