Skip to content

Conversation

@Nokse22
Copy link
Owner

@Nokse22 Nokse22 commented Jul 16, 2025

This needs python-tidal >= v0.8.4, but now this version doesn't load the home page

@nyabinary nyabinary mentioned this pull request Aug 10, 2025
@Nokse22 Nokse22 force-pushed the nokse/use-radio-mix branch from 5767365 to ff00e65 Compare August 19, 2025 09:35
@Nokse22 Nokse22 marked this pull request as ready for review August 19, 2025 09:35
@Nokse22 Nokse22 requested review from drafolin and nilathedragon and removed request for drafolin August 19, 2025 14:52
@drafolin
Copy link
Collaborator

Does 0.8.5 work with this? And does 0.8.5 load the home page?

Comment on lines +755 to +756
if parameter.get_string() == "":
return
Copy link
Collaborator

Choose a reason for hiding this comment

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

What are those for?

Copy link
Owner Author

Choose a reason for hiding this comment

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

In case the parameter of the action is empty it will not try to load the page with an empty id.

Copy link
Collaborator

Choose a reason for hiding this comment

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

In case the parameter of the action is empty

does that ever actually happen? and if it happens, will it pop off the stack or will it just load a blank page?

@Nokse22 Nokse22 requested a review from drafolin August 19, 2025 19:11
@Nokse22
Copy link
Owner Author

Nokse22 commented Aug 19, 2025

Does 0.8.5 work with this? And does 0.8.5 load the home page?

Yes and yes.

@nilathedragon
Copy link
Collaborator

For tracks that have a radio available, this seems to work. Artists too. Homepage is not loading for me, checking which tidal package version I have...

@nilathedragon
Copy link
Collaborator

Exception "Item type 'DEEP_LINK' not implemented"

@nilathedragon
Copy link
Collaborator

It is on 0.8.5 but probably not related to this PR specifically.

@nyabinary
Copy link

Exception "Item type 'DEEP_LINK' not implemented"

this got fixed upstream in the upcoming release I believe

@Nokse22
Copy link
Owner Author

Nokse22 commented Aug 19, 2025

Exception "Item type 'DEEP_LINK' not implemented"

this got fixed upstream in the upcoming release I believe

Yes, maybe I can patch it for the Flatpak package at least (new version released)

Then I'd merge this

@Nokse22 Nokse22 merged commit 885f83f into master Aug 19, 2025
12 checks passed
@nilathedragon
Copy link
Collaborator

I saw 0.8.6 was released, testing that...

@Nokse22
Copy link
Owner Author

Nokse22 commented Aug 19, 2025

I saw 0.8.6 was released, testing that...

It works for me

@drafolin
Copy link
Collaborator

drafolin commented Aug 19, 2025

works with 0.8.6 for me; will adapt pkgbuilds

@nilathedragon
Copy link
Collaborator

Works indee, will update nixpkgs

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.

5 participants