-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Conversation
There was a problem hiding this 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
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! |
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)"
} |
0afa2b1
to
88e8b52
Compare
@wukko reminder to review again pls |
88e8b52
to
845bad8
Compare
thought i was PRing to the |
There was a problem hiding this 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!
this PR adds and documents the
composer
,genre
andalbum_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
PS: invest in a formatter. please