Skip to content

Conversation

@daniele-athome
Copy link
Contributor

@daniele-athome daniele-athome commented Mar 27, 2025

This PR adds support for a subtitle field for MP3 (ID3v2 "TIT3") and ASF.

I couldn't find a reference for an MP4 tag.

Required by beetbox/beets#5696

@daniele-athome daniele-athome marked this pull request as ready for review March 27, 2025 10:30
@daniele-athome
Copy link
Contributor Author

daniele-athome commented Mar 27, 2025

Modifying the library.py file worked (added the new field to Item._fields, opened beetbox/beets#5696 for that), but the new field is detected (and stored) only after an import. Do I have to re-import all of my library? Is there a quicker, more selective way maybe?

@jackwilsdon
Copy link
Member

beet update might work.

@daniele-athome
Copy link
Contributor Author

Thanks. Unfortunately it doesn't detect it, not even after touching the file. I could script it (i.e. a beets command plugin or even a shell script with some piped magic), but I hoped there was a faster way :-)

@daniele-athome daniele-athome changed the title Support for subtitle tag (TIT3) Support for subtitle field (TIT3) Mar 27, 2025
@michaelkrieger
Copy link

BumPing this. It is well supported by Navidrome and would be amazing to get using it

“TIT3 Song subtitle/description for example "Live at Wembley" or "12 Inch Extended Mix"”

@semohr semohr merged commit 475ebfb into beetbox:master Sep 30, 2025
1 of 9 checks passed
@semohr
Copy link
Contributor

semohr commented Sep 30, 2025

I just fixed the CI and merged it. Seemed like a nobrainer to me.

Sorry for the long delay!

semohr added a commit that referenced this pull request Sep 30, 2025
This PR adds support for a subtitle field for MP3 (ID3v2 "TIT3") and
ASF. Reference for an MP4 tag missing.

Required by beetbox/beets#5696
semohr added a commit that referenced this pull request Sep 30, 2025
This PR adds support for a subtitle field for MP3 (ID3v2 "TIT3") and
ASF. Reference for an MP4 tag missing.

Required by beetbox/beets#5696
@daniele-athome
Copy link
Contributor Author

Thanks for merging! I rebased the PR in beets.

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

Successfully merging this pull request may close these issues.

4 participants