#1 master

Merged
djib merged 3 commits from sebastian404/PDMameUpdate:master into master 10 months ago

WIP Some suggested ‘fixes’ for you.

WIP Some suggested 'fixes' for you.
djib reviewed 10 months ago
Please provide an updated template.json. Also, could you update the config value to mame-directory with a dash?
sebastian404 commented 10 months ago
Poster

mame_directory is already defined in the template.json, it's used in fetch_local_torrents and update_torrents.

I put my torrents in a ‘MAME’ subfolder seperate from the other torrents, so I need to set download_dir, otherwise it uses transmission's default directory.

mame_directory is already defined in the template.json, it's used in fetch_local_torrents and update_torrents. I put my torrents in a 'MAME' subfolder seperate from the other torrents, so I need to set download_dir, otherwise it uses transmission's default directory.
djib commented 10 months ago
Owner

Sorry I meant sharing your updated config.template.json in your PR.

Sorry I meant sharing your updated `config.template.json` in your PR.
sebastian404 commented 10 months ago
Poster

I dont understand your request, ‘mame_directory’ is in config.template.json it is part of the original script

https://djib.fr/djib/PDMameUpdate/src/branch/master/config.template.json

it is the directory the scipt looks in for torrents in fetch_local_torrents, and then again in update_torrents to rename the directory.

I dont understand your request, 'mame_directory' is in config.template.json it is part of the original script https://djib.fr/djib/PDMameUpdate/src/branch/master/config.template.json it is the directory the scipt looks in for torrents in fetch_local_torrents, and then again in update_torrents to rename the directory.
djib commented 10 months ago
Owner

Well I'll be damned, I totally skipped that. Sorry for the trouble and thanks for contributing.

Well I'll be damned, I totally skipped that. Sorry for the trouble and thanks for contributing.
The pull request has been merged as aa4b653217.
Sign in to join this conversation.
No Label
No Milestone
No Assignees
2 Participants
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
Cancel
Save
There is no content yet.