search

ghomasHudson / Jellyfin-Auto-Collections

Automatically make jellyfin collections from IMDB, Letterboxd lists and more.
MIT License
102 stars 12 forks source link

Add Letterboxd Watchlists #39

Closed consier closed 5 months ago

consier commented 5 months ago

It would be great to add Letterboxd Watchlists. However, naming could become an issue when adding multiple watchlists as they are always named watchlist

2024-06-17 16:39:01.196 | INFO     | __main__:<module>:43 - Starting up
2024-06-17 16:39:01.196 | INFO     | __main__:<module>:44 - Starting initial run
2024-06-17 16:39:01.681 | INFO     | __main__:main:29 - 
2024-06-17 16:39:01.681 | INFO     | __main__:main:30 - 
2024-06-17 16:39:01.681 | INFO     | __main__:main:31 - Getting list info for plugin: letterboxd, list id: username/watchlist
Page number:  1
Traceback (most recent call last):
  File "/Jellyfin-Auto-Collections/main.py", line 45, in <module>
    main(config)
  File "/Jellyfin-Auto-Collections/main.py", line 32, in main
    list_info = plugins[plugin_name].get_list(list_id, config['plugins'][plugin_name])
  File "/Jellyfin-Auto-Collections/plugins/letterboxd.py", line 25, in get_list
    list_name = soup.find('h1', {'class': 'title-1 prettify'}).text
AttributeError: 'NoneType' object has no attribute 'text'
ghomasHudson commented 5 months ago

The watchlists are a different format from the main lists so will require some more substantional reworking to get functional. I'll have a go later this week.

samueljim commented 5 months ago

bump on this :) Let me see if i can work this out

samueljim commented 5 months ago

https://github.com/ghomasHudson/Jellyfin-Auto-Collections/pull/42/files here is support for watchlists