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

Add ytdl-sub-gui template #436

Merged
merged 1 commit into from
Nov 3, 2023
Merged

Add ytdl-sub-gui template #436

merged 1 commit into from
Nov 3, 2023

Conversation

jmbannon
Copy link
Contributor

@jmbannon jmbannon commented Nov 3, 2023

@nwithan8 nwithan8 requested a review from Roxedus November 3, 2023 17:25
@nwithan8
Copy link
Member

nwithan8 commented Nov 3, 2023

Nice app, sure beats my janky Bash script running on a cron job.

@nwithan8 nwithan8 merged commit 9dab634 into selfhosters:master Nov 3, 2023
1 check passed
@jmbannon
Copy link
Contributor Author

jmbannon commented Nov 3, 2023

Nice app, sure beats my janky Bash script running on a cron job.

Thanks! You should give it a spin on unraid and let me know how it goes (including the template) - I actually don't use unraid so feedback here would be great

@catduckgnaf
Copy link
Contributor

can we keep the same defaults for /media to aline with non gui version please? also encourage same unraid forum default.
I don't see a temp directory either.

@jmbannon
Copy link
Contributor Author

jmbannon commented Nov 3, 2023

#437 updates the old (headless) template with similar values in the gui version. Configs/subscriptions will use those new paths by default (/tv_shows, etc`) but we have to keep media for backwards compatibility

@catduckgnaf
Copy link
Contributor

Confusing, not needed, and not best practice. But OK.

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.

3 participants