Skip to content
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

Enhancements in Track Handling and Artist Recognition for Yandex Music #134

Merged
merged 15 commits into from
Mar 10, 2024

Conversation

BrokenFG
Copy link
Contributor

Added support for manually uploaded tracks (only those uploaded by the token owner available)

Implemented recognition of track links without specifying the album, e.g., https://music.yandex.ru/track/75370207

Now, if a track has multiple artists, all of them will be listed in the author field instead of just one.

…e token owner available)

Implemented recognition of track links without specifying the album, e.g., https://music.yandex.ru/track/75370207
Now, if a track has multiple artists, all of them will be listed in the author field instead of just one.
…e token owner available)

Implemented recognition of track links without specifying the album, e.g., https://music.yandex.ru/track/75370207
Now, if a track has multiple artists, all of them will be listed in the author field instead of just one.
@topi314
Copy link
Owner

topi314 commented Oct 24, 2023

@AgutinVBoy any chance you can check this out?

@AgutinVBoy
Copy link
Contributor

@AgutinVBoy any chance you can check this out?

Ok. In the day.

@topi314
Copy link
Owner

topi314 commented Dec 10, 2023

is this done? or what is the current status on this?
you never asked us to review again

@BrokenFG
Copy link
Contributor Author

@topi314 Sorry, was on vacation. Yes, its done in my understanding

@topi314
Copy link
Owner

topi314 commented Jan 23, 2024

your branch is 11 commits behind master, can you please first merge before I take a look?
BrokenFG/LavaSrc@master...topi314:LavaSrc:master

@BrokenFG
Copy link
Contributor Author

Done

Copy link
Owner

@topi314 topi314 left a comment

Choose a reason for hiding this comment

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

please also update the supported links in the readme https://github.com/topi314/LavaSrc#yandex-music-1

@BrokenFG
Copy link
Contributor Author

BrokenFG commented Feb 9, 2024

done

BrokenFG and others added 5 commits February 22, 2024 21:09
…exMusicSourceManager.java

Co-authored-by: Toπ <git@topi.wtf>
…exMusicSourceManager.java

Co-authored-by: Toπ <git@topi.wtf>
…exMusicSourceManager.java

Co-authored-by: Toπ <git@topi.wtf>
…exMusicSourceManager.java

Co-authored-by: Toπ <git@topi.wtf>
@BrokenFG BrokenFG requested a review from topi314 March 4, 2024 23:39
@topi314 topi314 merged commit 32b7986 into topi314:master Mar 10, 2024
@topi314
Copy link
Owner

topi314 commented Mar 10, 2024

alright thanks for the contribution
sorry that it took another 5 days to merge

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.

None yet

3 participants