search

kepstin / magicisrc

JS web tool to batch-submit ISRCs to MusicBrainz
18 stars 6 forks source link

Support pasting multiple isrcs. #1

Open kepstin opened 7 years ago

kepstin commented 7 years ago

There should be the ability to paste text containing a list of multiple ISRCs, so they can be assigned sequentially to tracks. This saves a lot of effort individually copy/pasting ISRCs into separate fields.

An enhancement would be to parse certain common formats, e.g. cue files, to assign isrcs to tracks smartly.

jesus2099 commented 7 years ago

The attention points are

:)

kepstin commented 7 years ago

Duplicate marking is actually already present (along with a bunch of other sanity checks - e.g. for all-zero codes, and codes with unassigned country codes)

kepstin commented 4 years ago

Prototype UI. I'm currently working on the actual parsing backend code.

Screenshot_2019-11-29 kepstin%u2019s MagicISRC (Beta)

jesus2099 commented 4 years ago

Hello Kepstin. Actually I did my own dirty script to allow mass paste but I don’t publicise it, as I noticed I often have to loose hours fixing some badly mass submitted ISRC where some editor did not check the alignment of tracks, or missing tracks, etc.

kepstin commented 3 years ago

I've done an initial pass of multiple ISRC paste which is live now on https://magicisrc-beta.kepstin.ca/ - I'd appreciate any feedback.

This current implementation is a simple spread-sheet like thing where any time there's a line break in the pasted input, it will move to the next field. So this is useful for copy/pasting out of spreadsheets or simple text files. It makes no attempt to only find ISRCs in the pasted text.

I would still like to add some sort of parsing support at some point to handle more complex cases.

ToadKing commented 3 years ago

I use @jesus2099's script for pasting in CUE sheets with ISRC fields, so the beta page doesn't quite work for my case.

An example cue file: sample.cue.txt

kepstin commented 3 years ago

Right. For cue sheets, I really want to get a proper parser in place, so it can do things like skip tracks without isrcs.

FWIW: I'm not going to deploy the beta paste feature on https://magicisrc.kepstin.ca/ until I've gotten a better parsing solution in place or jesus2099's userscript has been updated to be compatible.

jesus2099 commented 3 years ago

Thanks @kepstin, it looks great to manage tracks with no ISRC (skip track). Still missing the ISRC match in each line: https://github.com/jesus2099/konami-command/issues/611

kepstin commented 19 hours ago

I've added support for matching/extracting the IRSC in pasted text, and released the multiple paste feature in v20240922.0.

I would still like to add a proper parser for things like CUE sheets or ISRC logs at some point…

jesus2099 commented 17 hours ago

Thanks @kepstin! I see there is this commit from you: 72a09b7929d9264a6ab85c6d69ab9f527b74b9b9 It looks a lot more compact than my old https://github.com/kepstin/magicisrc/pull/2/files

I am eager to test it, next time!