podgrab
podgrab copied to clipboard
akhilrex
Episode path format
This should close #70 by replacing the "Append Date" and "Append Episode Number" with additional formatting tokens that can be combined however desired. It also adds support for path separators in the format to introduce additional folder structures. The show title is still used as the base directory.
Something like: %YYYY%/%YYYY%-%mm%/%YYYY%-%mm%-%dd% %EpisodeTitle% would result in a file saved to something like: /assets/This American Life/2021/2021-05/2021-05-28 Good Grief.mp3
I am a little tied up with work. A quick look into the changes seem to be fine. I would still want to test this locally once. Thanks for the work. Give me some time.
Since you have already worked on it maybe you could see if #95 can be accommodated in this as well.
Thanks and Regards
Akhil Gupta
On Tue, 8 Jun 2021, 6:52 pm Arno Hautala, @.***> wrote:
No problem. Let me know if there's anything I can do to help.
— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/akhilrex/podgrab/pull/108#issuecomment-856762464, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEMVPRUC772BWQ2PFFHTTTTRYKRFANCNFSM46DBE4LQ .
Ah, I considered that, didn't see an obvious way to specify the number of zeros, and figured I'd leave it for another branch. I think it'd be easy enough to use something like %EpisodeNumber:##% to specify the number of zeroes. I'll take a look.
Just pushed another commit with support for setting a minimum width for the episode number using %EpisodeNumber:X% to set X minimum digits. If the argument is missing or not parseable as a number, it'll default to no padding.
I built this branch, tested it, and it works. I appreciate the additional flexibility in naming, particularly the left-padded episode numbers. Note that I didn't attempt to merge upstream when building/testing though. Would be cool to see this get merged up.
