8000 api/soundcloud: add more metadata fields by nexpid · Pull Request #1313 · imputnet/cobalt · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

api/soundcloud: add more metadata fields #1313

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

Merged
merged 3 commits into from
Jun 8, 2025

Conversation

nexpid
Copy link
@nexpid nexpid commented Jun 2, 2025

this PR adds and documents the composer, genre and album_artist metadata tags. it also adds more metadata parsing to the soundcloud service:

  • album
  • album_artist
  • composer
  • genre
  • date
  • copyright

why?

having more metadata is always nice. this is a comparison with onthespot in foobar2000

onthespot develop feat/more-metadata
foobar2000_zfKvABMRfN foobar2000_ZNfxyQ3R1x foobar2000_ohvvmuCuLl

PS: invest in a formatter. please

8000

@nexpid
Copy link
Author
nexpid commented Jun 2, 2025

couldn't find a track with a composer but i pinky promise it gets parsed properly. here's another track which also has a genre specified:
image
soundcloud sadly (to my knowledge) doesn't tell you the track number of a song

Copy link
Member
@wukko wukko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add more optional chaining operators (?.), because else an error will be thrown instead of an empty metadata field

@wukko
Copy link
Member
wukko commented Jun 3, 2025

couldn't find a track with a composer

if you can't find it, then it's probably not there. i wouldn't add something you're not sure in existence of. you can always add it later!

@nexpid
Copy link
Author
nexpid commented Jun 3, 2025

if you can't find it, then it's probably not there

turns out I just didn't look hard enough, this song by Imogen Heap has proper credits

{
	"id": 1978104099,
	"urn": "soundcloud:tracks:1978104099",
	"artist": "Imogen Heap",
    // ........
	"writer_composer": "Imogen Heap, Guy Sigsworth",
	"release_title": "Let Go (on the hang in the park)"
}

@nexpid nexpid requested a review from wukko June 3, 2025 17:43
@nexpid nexpid force-pushed the feat/more-metadata branch from 0afa2b1 to 88e8b52 Compare June 3, 2025 17:44
@nexpid
Copy link
Author
nexpid commented Jun 5, 2025

@wukko reminder to review again pls

@nexpid nexpid force-pushed the feat/more-metadata branch from 88e8b52 to 845bad8 Compare June 7, 2025 20:08
@nexpid
Copy link
Author
nexpid commented Jun 7, 2025

thought i was PRing to the develop branch? oh well

@nexpid nexpid changed the base branch from main to develop June 7, 2025 20:17
@nexpid nexpid requested a review from wukko June 7, 2025 20:17
wukko
wukko approved these changes Jun 8, 2025
Copy link
Member
@wukko wukko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thank you for the pr!

@wukko wukko changed the title api/soundcloud: add more metadata support api/soundcloud: add more metadata fields Jun 8, 2025
@wukko wukko merged commit 170cf29 into imputnet:develop Jun 8, 2025
1 check passed
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.

3 participants
0