nick42d / youtui

TUI and API for YouTube Music written in Rust
MIT License
46 stars 4 forks source link

API error: no songs found #138

Closed rkratky closed 3 months ago

rkratky commented 3 months ago

I'm guessing another parsing problem. The error occurs after searching for "doors" and then pressing enter on the first search result ("The Doors"). image test.json

matheus-rosagaze commented 3 months ago

Yeah, I'm having the same issue. Open youtui, search for any artist, press enter, then there are no songs to listen.

Dunno exactly how to confirm if this is the error, but I find this: image

nick42d commented 3 months ago

Yeah, I'm having the same issue. Open youtui, search for any artist, press enter, then there are no songs to listen.

Dunno exactly how to confirm if this is the error, but I find this: image

Hi there @matheus-rosagaze, it looks like you might be on an older version of youtui. Could you please upgrade to latest version and test again?

nick42d commented 3 months ago

I'm guessing another parsing problem. The error occurs after searching for "doors" and then pressing enter on the first search result ("The Doors"). image test.json

Hi there @rkratky, thanks for providing the test file :) I can reproduce this using that file. It looks like you don't have the "Add To Library" button, which the API was expecting. I will make the add/remove from library tokens optional in the API. For future reference, it looks like if an entire album is already in your library in YouTube Music, you can't add/remove the individual songs. image

nick42d commented 3 months ago

Resolved in #139, I'll keep this open briefly for feedback on the other reported issue.

rkratky commented 3 months ago

@nick42d, thanks. The search problem is resolved, but the log now shows a new API error (no test.json generated though): image Not sure if it's worth opening a new issue.

nick42d commented 3 months ago

This looks similar enough, let's keep this one open. It's the same root cause of missing library management buttons. Apologies for missing this, I'll get onto a fix, but please bear with me as I'll first scan to see if there is anywhere else that this might occur.

nick42d commented 3 months ago

Resolved in #140

rkratky commented 3 months ago

Confirming fixed.