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

Not an issue #13

Closed
tnmdynamiq opened this issue Apr 2, 2024 · 14 comments
Closed

Not an issue #13

tnmdynamiq opened this issue Apr 2, 2024 · 14 comments
Labels
enhancement New feature or request

Comments

@tnmdynamiq
Copy link

Hey Patrick,

Sorry to bother you, but I don't know of any other way to contact you regarding a feature request. I was wondering if there's a way to incorporate a feature in the config.json to list multiple target URLs that I frequently download from. This would automate the selection process, removing the need to manually uncomment the URLs in the config, which can be a bit of a hassle. It would be a great convenience to just update the cookie when necessary and have "patreon-dl" cycle through the predefined list of URLs.

@patrickkfkan patrickkfkan added the enhancement New feature or request label Apr 2, 2024
@patrickkfkan
Copy link
Owner

I think maybe add an option to read URLs from a file?

@patrickkfkan
Copy link
Owner

Implemented in v1.3.0. Please test.

Closing this now as resolved. If you find an issue with the implementation, feel free to start a new issue.

@tnmdynamiq
Copy link
Author

Will make an attempt to do that today, Thank you so much!

@tnmdynamiq
Copy link
Author

Implemented in v1.3.0. Please test.

Closing this now as resolved. If you find an issue with the implementation, feel free to start a new issue.

Just wanted to clarify, are you saying rather than uncommenting each show as I download them, I can now perform the same action without manual intervention using a list? Normally I uncomment each show one by one in my config:

[downloader]
target.url = https://www.patreon.com/showname1/posts
# target.url = https://www.patreon.com/showname2/posts
# target.url = https://www.patreon.com/showname3/posts
# target.url = https://www.patreon.com/showname4/posts
# target.url = https://www.patreon.com/showname5/posts
# target.url = https://www.patreon.com/showname6/posts

use.status.cache = 1
no.prompt = 1
path.to.ffmpeg = /opt/homebrew/opt/ffmpeg@4/bin/ffmpeg

[output]
out.dir = /Volumes/SMH
campaign.dir.name.format = "{creator.vanity}[ - ]?{campaign.name}"
content.dir.name.format = "{content.id}[ - ]?{content.name}"
media.filename.format = "{media.filename}"

[include]
locked.content = 1
posts.with.media.type = audio
campaign.info = 1
content.info = 1
content.media = audio

[request]
max.retries = 3
max.concurrent = 10
min.time = 33

[logger.console]
enabled = 1
log.level = info
include.date.time = 1
include.level = 1
include.originator = 1
include.error.stack = 0
date.time.format = "mmm dd HH:MM:ss"
color = 1

Please provide an example of what I can change to make it check all of the shows at once if that's the case? PS, is there anyway I can donate to you?

@patrickkfkan
Copy link
Owner

Ok, so instead of this in your conf:

[downloader]
target.url = https://www.patreon.com/showname1/posts
# target.url = https://www.patreon.com/showname2/posts
# target.url = https://www.patreon.com/showname3/posts
# target.url = https://www.patreon.com/showname4/posts
# target.url = https://www.patreon.com/showname5/posts
# target.url = https://www.patreon.com/showname6/posts

You would do this:

target.url = "https://www.patreon.com/showname1/posts, https://www.patreon.com/showname2/posts, https://www.patreon.com/showname3/posts ..."

Ideally, the conf should support mult-line / multiple entries with same key, but it doesn't because of .ini style restrictions. This would make target.url quite lengthy if you have lots of URLs. So instead of having them all in one comma-separated line, you can have the following in your conf:

[downloader]
target.url = urls.txt

And then you have a separate file urls.txt:

https://www.patreon.com/showname2/posts
https://www.patreon.com/showname3/posts
https://www.patreon.com/showname4/posts
https://www.patreon.com/showname5/posts
https://www.patreon.com/showname6/posts

# You can still comment out lines here, e.g.:
# https://www.patreon.com/showname7/posts

If you want to donate, you can find the sponsor button on the main page of this repo. Thanks!

@tnmdynamiq
Copy link
Author

Ok, so instead of this in your conf:

[downloader]
target.url = https://www.patreon.com/showname1/posts
# target.url = https://www.patreon.com/showname2/posts
# target.url = https://www.patreon.com/showname3/posts
# target.url = https://www.patreon.com/showname4/posts
# target.url = https://www.patreon.com/showname5/posts
# target.url = https://www.patreon.com/showname6/posts

You would do this:

target.url = "https://www.patreon.com/showname1/posts, https://www.patreon.com/showname2/posts, https://www.patreon.com/showname3/posts ..."

Ideally, the conf should support mult-line / multiple entries with same key, but it doesn't because of .ini style restrictions. This would make target.url quite lengthy if you have lots of URLs. So instead of having them all in one comma-separated line, you can have the following in your conf:

[downloader]
target.url = urls.txt

And then you have a separate file urls.txt:

https://www.patreon.com/showname2/posts
https://www.patreon.com/showname3/posts
https://www.patreon.com/showname4/posts
https://www.patreon.com/showname5/posts
https://www.patreon.com/showname6/posts

# You can still comment out lines here, e.g.:
# https://www.patreon.com/showname7/posts

If you want to donate, you can find the sponsor button on the main page of this repo. Thanks!

Excellent Thank you so much!

@tnmdynamiq
Copy link
Author

Ok, so instead of this in your conf:

[downloader]
target.url = https://www.patreon.com/showname1/posts
# target.url = https://www.patreon.com/showname2/posts
# target.url = https://www.patreon.com/showname3/posts
# target.url = https://www.patreon.com/showname4/posts
# target.url = https://www.patreon.com/showname5/posts
# target.url = https://www.patreon.com/showname6/posts

You would do this:

target.url = "https://www.patreon.com/showname1/posts, https://www.patreon.com/showname2/posts, https://www.patreon.com/showname3/posts ..."

Ideally, the conf should support mult-line / multiple entries with same key, but it doesn't because of .ini style restrictions. This would make target.url quite lengthy if you have lots of URLs. So instead of having them all in one comma-separated line, you can have the following in your conf:

[downloader]
target.url = urls.txt

And then you have a separate file urls.txt:

https://www.patreon.com/showname2/posts
https://www.patreon.com/showname3/posts
https://www.patreon.com/showname4/posts
https://www.patreon.com/showname5/posts
https://www.patreon.com/showname6/posts

# You can still comment out lines here, e.g.:
# https://www.patreon.com/showname7/posts

If you want to donate, you can find the sponsor button on the main page of this repo. Thanks!

Excellent Thank you so much!

It seems to be working as intended, which is much appreciated thank you again! I would like to address some things that make it a little confusing if its ran unattended. So after each download cycle I see something like this usually info: PostDownloader: Total 0 / 77 posts processed (skipped: 30 redundant, 47 with unmet media type criteria) The issue is I only see it for the most post section/show downloaded and the log above it. Unless I am debugging it, I only really need that message to know what has been downloaded. Anyway to just get that message for each target url unless otherwise specified in the config?

@patrickkfkan
Copy link
Owner

I save log files, so for me it's pretty easy to just tail the created logs to get the summary for each URL. I can collate those lines for each target and print them out at the end. Will that work for you?

@tnmdynamiq
Copy link
Author

I save log files, so for me it's pretty easy to just tail the created logs to get the summary for each URL. I can collate those lines for each target and print them out at the end. Will that work for you?

Oh that's a good point, yeah that will work for me. Thank you

@patrickkfkan
Copy link
Owner

Updated v1.4.0. This version prints the summary for all target URLs at the end. Please test.

@tnmdynamiq
Copy link
Author

Updated v1.4.0. This version prints the summary for all target URLs at the end. Please test.

Will do shortly & keep you updated. Thank you!

@tnmdynamiq
Copy link
Author

Updated v1.4.0. This version prints the summary for all target URLs at the end. Please test.

Sorry for the late response, you nailed it! Thank you again, I sincerely appreciate it!

@patrickkfkan
Copy link
Owner

You're welcome, and thanks for the support!

@tnmdynamiq
Copy link
Author

You're welcome, and thanks for the support!

My pleasure, you deserve it! Sorry I didn't sooner. I meant too but had something come up and forgot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants