AmpliPi icon indicating copy to clipboard operation
AmpliPi copied to clipboard

Disallow saving Pandora stream configs without all fields completed

Open rtertiaer opened this issue 1 year ago • 3 comments

The UI will gleefully send off your request with no "name" field; the API rightfully rejects it. we can both handle the form better before it hits the API, and also handle the 422 Unprocessable Request response from the API.

rtertiaer avatar Dec 19 '23 20:12 rtertiaer

It seems a great deal of error handling is missing here; in part, this is simply because there are no facilities built in to this app to actually handle and display errors to users. Currently fussing with the best way to address that with this issue.

rtertiaer avatar Jan 02 '24 20:01 rtertiaer

also add a description field

rtertiaer avatar Jan 08 '24 15:01 rtertiaer

Related to our meeting from this morning, I think this issue is the one we were talking about this for. While you're already screwing around with the form this would be added to, here's some added directions: Add explanation for how to find a pandora station ID, even if you just use a mui tooltip that contains a short how-to with this image from the recently updated webapp guide

SteveMicroNova avatar Jan 08 '24 18:01 SteveMicroNova