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

lyricwiki-qt: Add support for chartlyrics.com. Closes: #1217 #149

Conversation

radioactiveman
Copy link
Member

lyricwiki-qt: Add support for chartlyrics.com. Closes: #1217

lyrics.ovh has been unreliable in the past months.
Meanwhile it is working again but still has issues at times.

Offer chartlyrics.com as an alternative. It is also not perfect
with no support for HTTPS and lyrics for new songs are missing.
Hoever having a choice is still better. And it is free for
non-commercial use, unlike many other sites.

See also:

@radioactiveman radioactiveman force-pushed the lyricwiki-add-chartlyrics-provider branch 2 times, most recently from 71b5781 to 74e4d26 Compare January 17, 2024 00:15
lyrics.ovh has been unreliable in the past months.
Meanwhile it is working again but still has issues at times.

Offer chartlyrics.com as an alternative. It is also not perfect
with no support for HTTPS and lyrics for new songs are missing.
Hoever having a choice is still better. And it is free for
non-commercial use, unlike many other sites.

See also:
- http://www.chartlyrics.com/api.aspx (Terms of Use)
- NTag/lyrics.ovh#15
- NTag/lyrics.ovh#17
@radioactiveman radioactiveman force-pushed the lyricwiki-add-chartlyrics-provider branch from 74e4d26 to 81cc000 Compare January 17, 2024 00:42
Copy link
Member

@jlindgren90 jlindgren90 left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks!

@radioactiveman radioactiveman merged commit 8da856e into audacious-media-player:master Jan 17, 2024
8 checks 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.

2 participants