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

beet update for selective fields #2229

Closed
dangmai opened this issue Oct 18, 2016 · 3 comments · Fixed by #2231
Closed

beet update for selective fields #2229

dangmai opened this issue Oct 18, 2016 · 3 comments · Fixed by #2231
Labels
feature features we would like to implement

Comments

@dangmai
Copy link
Contributor

dangmai commented Oct 18, 2016

Right now beet update doesn't have an option for updating only selective fields yet, and I think it'd be a good addition.

My use case for that is the following: I've manually updated the music metadata out of band for the Grouping tag using an external media player. However that media player also updates other tags that it's not supposed to, and now I want to update beet's backend with only the new Grouping information.

If this is something desirable, I can open a PR and start working on those. I'm thinking of adding a -f flags to specify which fields to update, and they can be comma separated.

@sampsyo sampsyo added the feature features we would like to implement label Oct 19, 2016
@sampsyo
Copy link
Member

sampsyo commented Oct 19, 2016

Yes! This sounds great. The request even sounds familiar—although I can't find another ticket that mentions the idea.

An -f flag to the update command sounds reasonable. But I'd slightly prefer a version where you just provide the -f flag multiple times over a comma-separated list.

@dangmai
Copy link
Contributor Author

dangmai commented Oct 19, 2016

Yup multiple -f is definitely better, I'll implement it that way.

Is this the ticket you're thinking about? https://groups.google.com/forum/#!topic/beets-users/o0hibIx3FSI

@sampsyo
Copy link
Member

sampsyo commented Oct 19, 2016

Yep, that was exactly it. Thanks for digging through the mailing list.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature features we would like to implement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants