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

[dvr] add "directory" option to auto and time recorder entries to override directory settings for matching recordings. #580

Closed
wants to merge 1 commit into from

Conversation

Jalle19
Copy link
Contributor

@Jalle19 Jalle19 commented Jan 7, 2015

Fixes:

https://tvheadend.org/issues/2123
https://tvheadend.org/issues/2448

This adds a "Directory" field to autorec and timerec entries. When this value is present, all recordings created by these rules will be stored in the specified directory, regardless of the recording profile's settings.

Haven't tested how the recordings show up when listed through HTSP (e.g. in Kodi), will do that later.

@perexg for review

@@ -51,6 +51,10 @@
<dd>
The name you've given to the rule, e.g. 'Stuff involving Jeremy Clarkson'.
<p>
<dt><b>Directory</b>
<dd>
The directory in which to save recordings to. Setting this will override any directory related settings from the DVR configuration. Useful for e.g. recording multiple different news broadcasts into one common directory.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"The sub-directory into which to save recordings. Setting this will override any sub-directory related settings such as the creation of separate directories per day, channel, or title"

@Jalle19
Copy link
Contributor Author

Jalle19 commented Jan 7, 2015

@ProfYaffle are you happy with the current help text?

override directory settings for matching recordings.

Useful for e.g. recording multiple different shows into a single
directory (e.g. news).

Fixes #2123, fixes #2448
@Jalle19
Copy link
Contributor Author

Jalle19 commented Jan 7, 2015

Squashed

@ProfYaffle
Copy link
Member

Fine by me - perfectly clear now.

@Jalle19
Copy link
Contributor Author

Jalle19 commented Jan 7, 2015

@perexg should be fine then

@perexg
Copy link
Contributor

perexg commented Jan 7, 2015

Merged. Thanks.

@perexg perexg closed this Jan 7, 2015
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.

None yet

3 participants