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

Add more options #1

Closed
SenorSmartyPants opened this issue Mar 20, 2022 · 3 comments
Closed

Add more options #1

SenorSmartyPants opened this issue Mar 20, 2022 · 3 comments

Comments

@SenorSmartyPants
Copy link
Contributor

Add more options

  • enable/disable output of desc.txt
  • enable/disable output of narrator.txt
  • enable/disable including narrator in artists field
@Neurrone
Copy link
Owner

Thanks for the suggestion.

I'm not opposed in principle to adding more configurability, but I'm trying to figure out how to do so without adding too many options.

In the case of enabling / disabling the writing of desc and reader.txt, that could be 2 booleans that control that behaviour. That's probably something I could implement quickly in the coming weekend or the week after next once I have a few mins.

Disabling inclusion of narrator in artists is something I'm unsure about, because this then opens the doors to requests for arbitrary mappings of metadata to tags. For example, that could be implemented as a simple boolean but would not be feasible in the long run to support more customization.

Customizing tag metadata like this will only happen later on, once i figure out how this should be configured.

@SenorSmartyPants
Copy link
Contributor Author

Glad to hear it about the txt files.

Understandable about the tags. I can always fork it to my taste. Or if this ever gets implemented beetbox/beets#4095

@SenorSmartyPants
Copy link
Contributor Author

These features were added in #16 and #17

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

No branches or pull requests

2 participants