Get home page using updated endpoint (Fixes #324)#352
Get home page using updated endpoint (Fixes #324)#352Nokse22 wants to merge 5 commits intoEbbLabs:mainfrom
Conversation
|
Now just mixes don't work |
|
Thanks for your additions. I've made a new branch here to continue working on it. I've made some adjustments and will see if i can get mixes working as well. |
Hmm. Mixes (Both v1 and v2) seem to work fine in my end (albeit slow to parse). What error do you get? |
|
The mixes on the home fail to parse, that's why I have commented these lines. The json is completely different Here is the same mix from the
|
|
Ah sorry, I see what went wrong before when I tested. It looks like I simply used the Mix id and parsed it directly. Of course that will work fine, but that is not the intended way (we should use the json obj instead to populate the object). And as you point out, the fields seem to be quite a bit different but it should be relatively straightforward to fix them. But it seems odd if the json object is changed only in this case. Perhaps the json object has been changed, thus parsing broken for some time? |
|
The rest of the mixes still return the old json and are parsed correctly, I don't know why it is this way. I also tried using the id, but it become very slow. |
|
Take a look: This seems to work as expected but could do with a bit of cleanup (I am not a fan of having both V1 and V2 parsers/endpoints). For Mixes, they could probably be removed altogether, as I am not sure why we prefer to keep the V1 endpoint in the first place. For Pages, I have not checked what endpoint is used when browsing other pages. |


This is just an hack, now only playlist works. Help is very much appreciated