search

BreakOutEvent / breakout-frontend

Web Frontend
GNU Affero General Public License v3.0
6 stars 2 forks source link

Change user credentials | #390 Passwort / E-Mail-Adresse im Profil ändern #410

Closed abatmas closed 2 years ago

sibbl commented 2 years ago

@SimonAbat ich würde mich freuen wenn du nochmal drüber schauen kannst.

Neu ist eigentlich nur dass ich von den zwei E-Mail-Feldern wieder weg gegangen bin (unter dem Input-Feld steht "Bestätigung von ausstehend") und beim Speichern des Passworts noch eine konkrete Info zum Fehlschlagen gebe, wenn das aktuelle Passwort falsch ist.

Ich denke damit sollte das auch aus Nutzersicht verständlicher sein :)

sibbl commented 2 years ago

Da der Build aktuell wegen npm ci failed, würde ich vorschlagen erstmal die Cookie-Banners zu mergen. Dort wurden Packages hinzugefügt/entfernt, was auch das Problem hier lösen dürfte.

Vorteil: vermeidet einen Merge Konflikt wenn wir das jetzt hier nochmal parallel machen.

Genaue Fehlermeldung: "npm ERR! npm ci can only install packages when your package.json and package-lock.json or npm-shrinkwrap.json are in sync. Please update your lock file with npm install before continuing."