Skip to content

Allow searching by album.#15

Open
Priva28 wants to merge 2 commits intoddddxxx:masterfrom
Priva28:allow-searching-by-album
Open

Allow searching by album.#15
Priva28 wants to merge 2 commits intoddddxxx:masterfrom
Priva28:allow-searching-by-album

Conversation

@Priva28
Copy link

@Priva28 Priva28 commented May 5, 2022

Sometimes I have found that there are songs (or singles with features for example) that have slightly different lyrics or a different timing.

My change allows the application to specify an album as part of the search request:

  • There is now an optional album property that you can add when creating a .info search request.
  • The NetEase and QQ searches will append the album name at the end of the search query.
  • The quality of the returned lyrics will be increased if the album is matching.

I have tested and verified that adding the album to the search request significantly narrows the results and increases the quality value for proper matching results which makes it easier to deliver accurate lyrics to the user.

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.

1 participant