OxygenCobalt / Auxio

A simple, rational music player for android
GNU General Public License v3.0
1.78k stars 117 forks source link

Give reasons in error message when importing a playlist fails #744

Closed edwinst closed 1 month ago

edwinst commented 1 month ago

Description

The app current gives a very general message like "playlist could not be imported" if there are any problems when importing a playlist. It would help a lot with troubleshooting if the app gave the reason why the playlist could not be imported. By trial and error I found the following problems in my playlist files:

Problem solved

The problem is that currently the user has to guess what is wrong with a playlist file if the import fails.

Other implementations

I don't think other implementations are relevant here, as the details of the error messages really depend on the Auxio import code.

Benefit

Giving reasons in the error message speeds up troubleshooting, improves user experience, and reduces need for user support.

Duplicates

OxygenCobalt commented 1 month ago

This is kind of a duplicate of #739. The issue is that this requires a more complicated indicator (i.e a Snackbar), which means another view component I have to cram into the main UI. This is frustrating since I have no clue what will actually happen. Merging and adding this to the issue.