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

Added LavaSrc pluginInfo for yandex music tracks #191

Closed
wants to merge 3 commits into from

Conversation

Krispeckt
Copy link
Contributor

No description provided.

@Krispeckt
Copy link
Contributor Author

Fixed it

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.

while you are at it are you also interested in adding the ExtendedAudioPlaylist stuff?
this is not a requirement tho

this.sourceManager = sourceManager;
}

public YandexMusicAudioTrack(AudioTrackInfo trackInfo, String albumName, String albumUrl, String artistUrl, String artistArtworkUrl, String previewUrl, YandexMusicSourceManager sourceManager) {
Copy link
Owner

Choose a reason for hiding this comment

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

since previewUrl is always null it seems like let's remove it from the constructor too like isPreview

@topi314 topi314 added the enhancement New feature or request label May 13, 2024
- Fixed bugs when getting links in playlists and some other things
@Krispeckt
Copy link
Contributor Author

while you are at it are you also interested in adding the ExtendedAudioPlaylist stuff? this is not a requirement tho

I don't get it, it's already implemented :/ But there were a few bugs there

@Krispeckt Krispeckt deleted the branch topi314:master May 14, 2024 15:56
@Krispeckt Krispeckt closed this May 14, 2024
@Krispeckt Krispeckt deleted the master branch May 14, 2024 15:56
@Krispeckt Krispeckt restored the master branch May 14, 2024 15:56
@Krispeckt Krispeckt deleted the master branch May 14, 2024 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants