Skip to content

Set parameter "TV_DOWNLOAD_DIR" as default input value#722

Closed
qrioniclabs wants to merge 1 commit intomidgetspy:masterfrom
qrioniclabs:master
Closed

Set parameter "TV_DOWNLOAD_DIR" as default input value#722
qrioniclabs wants to merge 1 commit intomidgetspy:masterfrom
qrioniclabs:master

Conversation

@qrioniclabs
Copy link
Copy Markdown

This little change sets the default input value of the "Manual Post-Processing" interface to the TV Download Dir from Post-Processing Settings. This helps doing a quick rescan of the default directory, e.g. after repairing a failed download.

This little change sets the default input value of the "Manual Post-Processing" interface to the TV Download Dir from Post-Processing Settings. This helps doing a quick rescan of the default directory, e.g. after repairing a failed download.
@thezoggy
Copy link
Copy Markdown
Contributor

thezoggy commented Sep 7, 2013

the default input value is stored via localStorage/cookie once its been successfully pp. but anyways this change is already in itozofo's branch of his post-processing changes..

@qrioniclabs
Copy link
Copy Markdown
Author

@thezoggy still, isn't this a valuable addition? Sorry, I'm relatively new to collaborating on github. Also, I am unable to find that branch of itozofo's, you're talking about... Thanks ;-)

@thezoggy
Copy link
Copy Markdown
Contributor

thezoggy commented Sep 9, 2013

his branch: https://github.com/itofzo/Sick-Beard/tree/pre_change_postprocess_replace_rules_dev

  • Add tv download dir as default manual post-process directory

@qrioniclabs
Copy link
Copy Markdown
Author

Well as said before I'm new to this and don't know the rules, but as far as I can see it, itofzo added his commit after I did mine: https://github.com/itofzo/Sick-Beard/commits/9f3ea3e95fb501fb2133d66e83d98ad5d94d3d66/data/interfaces/default/home_postprocess.tmpl

itofzo: Sep 06, 2013
this: Sep 04, 2013

@thezoggy
Copy link
Copy Markdown
Contributor

i show itofzo's change is 11 days ago while yours is 5 days ago... but regardless no biggie. if midget wants to pull your change first thats fine.. the problem with your submission is that new code goes into the development branch first.
my main point was i just simply noting that this change ive already seen in one of itofzo's recent branches :)

@ghost ghost closed this in 9f309a4 Mar 9, 2014
gborri pushed a commit to gborri/Sick-Beard that referenced this pull request Aug 20, 2014
This pull request was closed.
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.

2 participants