Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UI: Better API errors #11602

Merged
merged 2 commits into from
Jan 13, 2024
Merged

UI: Better API errors #11602

merged 2 commits into from
Jan 13, 2024

Conversation

naltatis
Copy link
Member

fixes #11471

  • 📝 multiline error messages in notification modal
  • 📯 show HTTP method on api errors
  • 📧 show HTTP response text if exists

A few examples

Bildschirmfoto 2024-01-10 um 16 30 35

Bildschirmfoto 2024-01-10 um 16 31 30

Bildschirmfoto 2024-01-10 um 16 34 26

@naltatis naltatis requested a review from andig January 10, 2024 15:38
@naltatis naltatis added enhancement New feature or request ux User experience/ interface labels Jan 10, 2024
@naltatis naltatis changed the title UI: Show error response UI: Better HTTP errors Jan 10, 2024
@naltatis naltatis changed the title UI: Better HTTP errors UI: Better API errors Jan 10, 2024
@andig
Copy link
Member

andig commented Jan 10, 2024

Sollen wir die Reihenfolge nich ändern? Es kommt ja erst der Request und dann das Ergebnis? Ist ohnehin ein technischer Fehler der egtl. Nicht passieren sollte.

@naltatis
Copy link
Member Author

Ich würds so lassen. Die erste Zeile bekommen wir immer. Zweite und dritte sind optional. Geht er Request bspw verloren oder wird geblockt kann's sein, dass die dann nicht da sind. So ist's technisch am einfachsten.

Und ja, eigentlich wollen wir hier keine technischen sondern aussagekräftige, übersetzte fachliche Fehler anzeigen. Aber das ist noch ne andere Baustelle 😃

@andig
Copy link
Member

andig commented Jan 10, 2024

Die erste Zeile bekommen wir immer. Zweite und dritte sind optional.

Die erste Zeile kannst Du in dem Fall nur bekommen, wenn es die Dritte vorher gab ;)

@GrimmiMeloni
Copy link
Sponsor Collaborator

Die erste Zeile bekommen wir immer. Zweite und dritte sind optional.

Die erste Zeile kannst Du in dem Fall nur bekommen, wenn es die Dritte vorher gab ;)

Richtig, aber die erste ist die mit dem Interessanten drin. Deswegen gehört die nach oben. #frechVonDerSeiteReinGerufen

@andig
Copy link
Member

andig commented Jan 13, 2024

Richtig, aber die erste ist die mit dem Interessanten drin. Deswegen gehört die nach oben. #frechVonDerSeiteReinGerufen

Das sehe ich komplett anders. Für mich als Anwender ist ein HTTP 400 völlig Banane. Wenn der Server dazu eine sinnvolle Nachricht schickt interessiert mich das.

@andig andig merged commit d69b07c into master Jan 13, 2024
6 checks passed
@andig andig deleted the feature/api_error_response branch January 13, 2024 09:41
@GrimmiMeloni
Copy link
Sponsor Collaborator

Das sehe ich komplett anders. Für mich als Anwender ist ein HTTP 400 völlig Banane. Wenn der Server dazu eine sinnvolle Nachricht schickt interessiert mich das.

Stimmt auch wieder. Je nachdem aus welcher Perspektive man drauf schaut. 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ux User experience/ interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

UI: log technical error messages
3 participants