imimap / imimap21

imimap - redone in MEVN
https://showtime.f4.htw-berlin.de/ss21/master/m6-migrate-the-imimap-to-mevn/
MIT License
2 stars 1 forks source link

Speichern eines Praktikums nicht erlauben, wenn benötigte Felder nicht korrekt ausgefüllt wurden #295

Closed JuliaZamaitat closed 1 year ago

JuliaZamaitat commented 2 years ago

Man kann ein Praktikum speichern und nur das Firma Feld ist als Pflichtfeld markiert, obwohl mehrere Felder für das korrekte Funktionieren benötigt werden. Entsprechend die Felder markieren und Speichern verhindern, wenn dort keine Angaben sind.

JuliaZamaitat commented 2 years ago

Anmerkung: Anscheinend gibt es diese requested Fields, die aber nur requested sind, um den Praktikumsstatus von "planned" auf "requested" zu bringen (worüber es auch noch keine Transparenz für den User gibt):

export const requiredFields = [
  "startDate",
  "endDate",
  "operationalArea",
  "tasks",
  "workingHoursPerWeek",
  "supervisor.fullName",
  "supervisor.emailAddress",
];

User auf jeden Fall darüber benachrichtigen Lösung für die Fehler, die geworfen werden wenn man Felder erst frei lässt, finden.

JuliaZamaitat commented 2 years ago

Beim Eintragen der Firma sollte auch auf jeden Fall ein Dropdown erscheinen, das man auswählen kann, wenn man eine schon bestehende Firma auswählen will. Die jetzige Lösung, wo einfach stillschweigend eine bereits existierende übernommen wird, und wenn man es etwas anders schreibt, man eine neue anlegen muss, ist nicht wirklich gut.