[BUGFIX] Do not set music_directory in prebuilt presets (#783)

New music prebuilt presets had set `music_directory`, which is bad practice since it's not visible to the user which directory it would use. This forces `music_directory` to be set either in the user's config or subscription file
This commit is contained in:
Jesse Bannon 2023-10-25 11:36:51 -07:00 committed by GitHub
parent 852347dc7e
commit 11be119261
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 1 deletions

View file

@ -32,6 +32,9 @@ presets:
- "{track_genre}"
overrides:
# MUST DEFINE:
# music_directory
# Subscription overrides
subscription_indent_1: "Unset" # genre
url: "{subscription_value}"
@ -48,7 +51,6 @@ presets:
track_genre: "{subscription_indent_1}"
# Directory Overrides
music_directory: "/music"
artist_dir: "{track_artist_sanitized}"
album_dir: "[{track_year}] {track_album_sanitized}"
track_file_name: "{track_number_padded} - {track_title_sanitized}.{ext}"

View file

@ -360,6 +360,7 @@ class TestPrebuiltMusicPresets:
music_preset,
],
"overrides": {
"music_directory": "/music",
"subscription_value": "https://your.name.here",
},
},