CuBoulder / tiamat-theme

University of Colorado Boulder Drupal 9+ base theme
GNU General Public License v2.0
3 stars 0 forks source link

Display error messages to editors only #1392

Open kevincrafts opened 1 month ago

kevincrafts commented 1 month ago

Can we explore ways to display the various error messages, such as "Error retrieving people from the API endpoint. Please try again later." to site editors only? Hide them with CSS if you don't have an editor role maybe?

These error messages are of little help to the editors. In this case I believe the error is due to having too many filters selected, so it's never going to work no matter how much they try again later.

But the errors also give a bad impression to site visitors, where error messages across a site are publicly displayed. In these cases I believe it's better to just show nothing.

image
timurtripp commented 1 month ago

It's up to the editors to properly configure their site so errors don't display unless the user is on an unreliable internet connection or in an area of poor cell service, and the user genuinely needs to "try again later" (refresh the page when on a better connection). On the Faculty Publications block I also included a retry button for this purpose.

Alternatives to an error message could be:

The particular error in the screenshot shouldn't appear if the filters are simply too specific/not returning any people, so if it is then that would be a bug. It's worth discussing this more as a group.

timurtripp commented 1 month ago

Using the example of the Faculty Publications block, if the filters are too specific or incorrectly entered it says simply "There were no publications returned." and it's up to the site editor to fix that. "There was a problem fetching the publication data. Please try again later." and the retry button should only appear if the user's internet connection is bad or the host is experiencing temporary reliability issues.