Process server errors in API components #418
Conversation
There was a problem hiding this comment.
Sieht weitgehend sehr gut aus! Habe noch ein paar Inputs bzw. offene Fragen.
Noch ein visueller Bug, vielleicht related zu #420 oder einem versuchten Fix dafür?
Ich habe noch ein Ticket erfasst zu einem Problem das mir beim manuellen Testing aufgefallen ist. Die Error-Message unter dem Input-Feld funktioniert aber korrekt.
|
|
||
| // 422 validation error | ||
| if (error.name === 'ServerException' && error.response && error.response.status === 422) { | ||
| this.serverErrorMessage = 'Validation error: ' |
There was a problem hiding this comment.
Weshalb schreiben wir das in die serverErrorMessage und nicht in eine validationErrorMessage? Oder wieso heisst es nicht allgemeiner errorMessage und loading Fehler sind auch gleich drin?
There was a problem hiding this comment.
Hmm, errorMessages gibt es ja als computed und dort werden die einzelnen Messages zusammengebaut (als Array von mehreren Messages, nach Konzept Vuetify wird aber nur jeweils die erste Message angezeigt).
validationErrorMessage würde ich nicht nehmen um das nicht mit der Frontend Validation zu vermischen. Es heisst zwar beides "Validation" hat aber ja einen ganz anderen Ursprung.
Loading und (patch) serverError könnte man schon zusammen nehmen, ich seh jetzt aber nicht den grossen Benefit. Wär es für dich klarer wenn alles in einer serverErrorMessage drin ist, egal ob vom load oder patch?
There was a problem hiding this comment.
Meine Idee ging schon eher in die Richtung Frontend- und Backend-Validierung am gleichen Ort kombinieren, da die beiden sich ja theoretisch zumindest nie widersprechen sollten. Aber ich verstehe auch wenn das nicht so einfach ist / wenn die nicht wirklich zusammenpassen.
Ist auch gar nicht so schlimm so wie du es aktuell gelöst hast, einfach etwas unintuitiv dass zuerst die Server-Validierung (User ist der Grund) und Server-Fehler (Server ist der Grund) an einem Ort (serverErrorMessage) zusammengefasst werden, und nachher erst mit der Frontend-Validierung und Loading-Fehlern in ein gemeinsames computed errorMessage gestopft werden
|
@usu An welchem Beispiel kann ich mir die Server Errors anschauen? |
@manuelmeister In der |
Fixes #275 (sort of)
Catch