-
-
Notifications
You must be signed in to change notification settings - Fork 16
Functionality to disable specific lyrics provider #54
Conversation
A couple points to this:
The PR is not mergeable in this state; please fix all those issues and then I will review it once more. |
|
The Please take a look at the review I started. |
me using snakecases as default make sense. cuz I come from Python Background. and we only use snakecase there. |
I want to know if this PR works as advertised in this state... |
it does. tested and worked. |
WHY? It is meant to inform the user that lyrics aren't available! |
So, IDK if that was my fault or not, but the metadata query code was faulty lol. Anyway, I fixed it now, and also the list now reorders the services' priority. |
kinda closes #53