c4a8-web / shared-components

Storybook driven shared web components.
1 stars 2 forks source link

Fehlermeldungen im Bewerbungsformular #103

Closed Listor closed 8 months ago

Listor commented 11 months ago

Als User möchte ich auf Fehler beim Versenden meiner Bewerbung hingewiesen weredn.

Acceptance criteria:

Infos: Herausfinden, welche Fehlermeldungen von der Schnittstelle kommen und bestimmen in welchem Fall, welche Fehlermeldung ausgeben werden soll.

Listor commented 10 months ago

Fehlermeldungen beim Dokumenten-Upload:

Fehlermeldungen beim Absenden der Bewerbung:

Ich würde die Fehlermeldungen 400, 401, 403, 422, 429 und 500 alle mit einem generischen Fehler ausspielen. Der User hat hier nicht wirklich Einfluss darauf, ob der Request abgesendet werden kann. Dies kann auf Fehler in unserer Implementierung oder bei Personio hindeuten. Auch Too Many Requests, können wir dem User nicht sagen, wann er wieder einen Request abschicken darf.

Image

Dafür dann das generische Erdmännchen, anstatt der Rakete wie hier https://c4a8-web.github.io/shared-components/?path=/story/components-modal-error--modal-error . Der User kann hier Nichts mehr machen, aus meiner Perspektive.

Fehlermeldungen für Felder, gibt die Schnittstelle nicht raus laus Dokumentation, also gibt es auch Nichts, das wir dann behandeln können.

Bei 413 könnten wir dem User mitteilen, dass seine Datei zu groß ist, obwohl unser Frontend darauf prüft, gibt das keine 100%ige Sicherheit. Kann also sein, dass der Server das Trotzdem ausgibt.

Mein Vorschlag wäre hier dann den Fehler am Feld anzuzeigen mit der Meldung. "Eine oder mehrere Dateien sind zu groß.". Das ist auch der Frontend Validierungstext, der im Moment angezeigt wird und schon in der Übersetzung drin ist.

Listor commented 10 months ago

@cakageka @KristinRolfes was denkt ihr, soll ich das so umsetzen?

KristinRolfes commented 10 months ago

Wie gerade besprochen:

Listor commented 9 months ago

So würde die generische Fehlermeldung dann aussehen @KristinRolfes

Image

KristinRolfes commented 9 months ago

@Listor, fnde ich gut!

cakageka commented 9 months ago

Da würde man sich doch wünschen, dass die Fehlermeldung öfter kommt :-)

cakageka commented 9 months ago

Text müssten wir allerdings ändern:

Etwas ist schief gelaufen!

Bitte versuche es später noch einmal oder kontaktiere uns unter:

info@glueckkanja.com +49 69 4005520

Listor commented 9 months ago

Ja ist nur Platzhalter, aber ich nehm den Text mit in die Änderungen.

Listor commented 9 months ago

Die generische Fehlermeldung kann man jetzt hier testen https://c4a8-web.github.io/shared-components/?path=/story/components-job-detail--error-generic

Die Fehlermeldung mit der Dateigröße kann man leider nicht in den Shared-Components statisch testen, nur mit einem echten laufendem Storybook, dass quasi für manche Fälle einen "Server" simuliert.

Die Fehlermeldung sieht so aus, wie wenn man selber eine zu große Datei hochlädt, nur diesmal kommt die Antwort vom Server. Den Testfall kann ich auch nicht im Live System herstellen, weil ich die Schnittstelle von Personio nicht dazu zwingen kann, mir dieses Ergebnis zu liefern.

Macht das Feature relativ schwer bis unmöglich zu testen :/

Hier sieht man das Frontend schlägt bei 35.1KB nicht fehl, schickt also den Request an mein Fake Backend, das sagt dann aber, die Datei ist doch zu groß, und man landet dann hier und könnte ggf. seine Datei noch leicht verkleinern. Der Fehler sollte generell kaum auftreten, könnte mir nur vorstellen, dass die Komma Zahlen unterm Strich, die wir im Frontend errechnen, nicht 100% mit dem vom Backend übereinstimmen und dadurch so ein Fehler auftreten könnte.

Image

@cakageka ist die Frage, ob dir das als "Test" reicht oder ob wir Zeit investieren wollen, diesen Fall testbar zu machen?

cakageka commented 8 months ago

das passt.