-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Let mbsubmit plugin open seeded "add release" page #1866
Comments
There's actually no CLI interface to the plugin. Basically, all it does is:
except as an option at the importer prompts. |
I see. |
So the workflow is:
Update: Hm, that didn’t work. |
Whoa; crazy. Maybe we should steal that feature. 😃 Yes, that's approximately my understanding. It might be worth getting feedback from @awesomer and @diego-plan9, who are more familiar with the plugin (@diego-plan9 wrote it 😃). |
Hm, the question is how the workflow should look; My idea:
For reference, here is the Picard plugin that does something similar: https://github.com/musicbrainz/picard/blob/master/contrib/plugins/addrelease.py |
The workflow for the current mbsubmit plugin is :
The whole point of mbsubmit is to prevent you from having to import the files "As Is" in order to "print" their tracklist. |
Oops; sorry for misunderstanding your comment, @Profpatsch. Yes, that sounds like a good plan. FWIW, the |
Thanks for "discovering" the picard auto-magically-upload feature, @Profpatsch ! It's a rather neat idea, and if we can assume that the MusicBrainz UI does not change too much and it does not go against MusicBrainz policies (which I'm assuming won't be a problem since it's included in picard?), I feel it would be worth "stealing" it indeed! As for the current usage of the plugin, it's basically as described by @awesomer at #1866 (comment) (please let me know if the existing documentation did not make a good job for explaining it, as it might as well be the case). It seems that if we manage to bring in the feature from picard, we would only need to change step 2 (so instead of printing it performs the actual submission) - would that suit your needs? I'm mainly curious about if the third step on your proposed workflow (the separate |
Oho, I didn’t know about that. Probably because I did a lazy search for
Yes, that would be perfect. Then we could call the option
Not sure if that is required then. Maybe it’s best to keep the amount of command line subcommands to an absolute minimum. |
Gotcha, thanks for sharing your thoughts on the workflow! I'm actually quite tempted to work on bringing the form-based musicbrainz submitting to beets, as it will greatly enhance the usefulness of the |
@diego-plan9 Great! |
Hey all :), For reference, here's picard's "Add Cluster As Release" plugin implementation of this feature: https://github.com/metabrainz/picard-plugins/blob/2.0/plugins/addrelease/addrelease.py Source: https://community.metabrainz.org/t/a-way-to-batch-import-data/327361/ |
Hey all. In #4807 I added a rather simple addition to |
I wasn't aware there's an issue for this, PR here #5300 |
When packaging I found mbsubmit, but I can’t figure out how to access it’s functionality from the command line.
The text was updated successfully, but these errors were encountered: