-
Beta Was this translation helpful? Give feedback.
Answered by
OzGav
Feb 25, 2026
Replies: 2 comments 5 replies
-
|
I do see a "radios" folder under Browse - Music Assistant - radios, but there are not any contents, nor a button to add a URL: |
Beta Was this translation helpful? Give feedback.
4 replies
-
|
"it isn’t practical to refresh pages when this completes as we can’t know
what page the user is then on"
If someone adds a radio station URL using the button in the list of radio
stations, doesn't that itself tell us what page the user is on? AFAIK,
there isn't another way to do that, so I guess I don't see the ambiguity.
For example, when a user adds a new provider, the provider is added, and
the user immediately sees a list of providers that has been updated with
the new one, without having to click to refresh.
When a user adds a new playlist, the list of playlists is immediately
updated and the user sees that the new one has been added without having to
click to refresh.
Is there something unique about adding a radio station URL that makes it
impossible for the app to display that same behavior when adding something
there?
…On Wed, Feb 25, 2026 at 3:53 PM OzGav ***@***.***> wrote:
I think the proper thing to do if there are no radio stations is to show
the Radio page anyway, and if there are no items, display text explaining
how to add items.
I just said we are changing this.
Yes you do have to refresh the UI because additions to the database happen
in the background and it isn’t practical to refresh pages when this
completes as we can’t know what page the user is then on.
Also rather than being critical you can make PRs to the documentation or
be constructive instead of saying “generally to be poor”.
—
Reply to this email directly, view it on GitHub
<#4998 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AETK6XZ7UXOBQ6A67RKODAD4NYYXHAVCNFSM6AAAAACV7XRSO2VHI2DSMVQWIX3LMV43URDJONRXK43TNFXW4Q3PNVWWK3TUHMYTKOJSHAZTGOI>
.
You are receiving this because you authored the thread.Message ID:
***@***.***
com>
|
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment



Update: I have just submitted a PR to change this behaviour. So expect that to flow through to the various server versions over the next couple of weeks.