mirror of
https://github.com/tubearchivist/tubearchivist-jf.git
synced 2026-03-24 05:26:49 +00:00
[GH-ISSUE #19] Script does not do anything #13
Labels
No labels
duplicate
enhancement
help wanted
help wanted
invalid
pull-request
question
question
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
tubearchivist/tubearchivist-jf#13
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @CommanderRedYT on GitHub (Nov 10, 2023).
Original GitHub issue: https://github.com/tubearchivist/tubearchivist-jf/issues/19
My setup is 90% the same (with the exception of jellyfin running outside of docker). When I run the manual start function, the script prints few things, but my library remains completely the same.
The titles are not changed, no thumbnails are there, the only thing that is visible is the youtube video description.
@bbilly1 commented on GitHub (Nov 12, 2023):
Maybe some of these days we can all reunite under a tree in nature somewhere, can all come to an understanding that we don't have magic, no clairvoyant abilities, no way and no will to guess what you could possibly see in the logs. Some day.
@CommanderRedYT commented on GitHub (Nov 12, 2023):
Maybe some of these days we can get actual log output (for example, the "Found Folder" is something I had to patch because this wonderful tool did not have enough logging)
@bbilly1 commented on GitHub (Nov 15, 2023):
There is not enough info to do anything here. You probably want to stick with the documented docker compose setup.
@CommanderRedYT commented on GitHub (Nov 15, 2023):
Okay, I will rewrite all of this, having a media server inside docker is a no-go, especially with a setup that already exists.
@bbilly1 commented on GitHub (Nov 15, 2023):
None the less, this integration is tested functions as documented, not necessarily as you have your things setup. If you want to help improve this to make this compatible with other setups, please make a PR with your changes.
@CommanderRedYT commented on GitHub (Nov 15, 2023):
While debugging, I found that https://github.com/tubearchivist/tubearchivist-jf/blob/master/app/src/series.py#L51 returns a empty array. When I looked at the ID in the query, it looked different than the ID returned in the constructor. I will now trace back why this is the case.
@CommanderRedYT commented on GitHub (Nov 15, 2023):
I have traced this issue back to that my folder on my physical disk is not called "YouTube", but "TubeArchivist". I have another YouTube folder for my previous YouTube download setup. I will make a PR for having a custom folder name
@bbilly1 commented on GitHub (Nov 15, 2023):
Good to hear you found it. No need to make this configurable, that just add unneeded complexity. Just name it as documented and you should be good.
@CommanderRedYT commented on GitHub (Nov 15, 2023):
I made a PR, because renaming is not a option for me as too many things depend on the name. I am sure that I am not the only one. PR: https://github.com/tubearchivist/tubearchivist-jf/pull/20