merge request id	219
Titel	Benchrichtigungen bei User-Änderung - Feature/ll 481 notify user on core data change
Beschreibung	Wird bei einem User das PW geändert (später auch: Passwort wurde invalidiert) oder die Mailadresse angepasst, so soll eine Benachrichtigung an den betreffenden User gesendet werden Ebenfalls enthalten sind LL-482 und LL-483
Status	merged
Created at	2020-07-17T13:39:36.640Z
Updated at	2020-07-28T06:48:58.380Z
Source branch	feature/LL-481-notify-user-on-core-data-change
Target branch	develop
Author	Gregor Gabriel
Assignee	Sascha Immig

Discussion notes

Warum hin und her wechseln? können wir nicht `$old_email`und `$new_email` an `UserMailModified()` übergeben, statt `$this->email` zu manipulieren? Das erscheint mir einleuchtender... Order nutzen wir die erneute Zuweisung von `$old_email` um an die alte Mailadresse zu senden?	Sascha Immig	2020-07-27T08:33:59.813Z
Das Problem ist, dass $this (also der User selbst) die Notification UserMailModified bekommt. Wenn überhaupt, dann müsste die Methode notify einen optionalen Parameter haben über den man die E-Mail-Adresse überschreiben kann oder man baut für diesen Zweckk eine von User erbende Klasse. Oder man fragt Marcel wie das eleganter geht ;-)	Gregor Gabriel	2020-07-27T08:55:48.076Z
evtl. kann ich wie in https://laravel.com/docs/7.x/notifications#using-the-notifiable-trait beschrieben die Methode routeNotificationForMail($notification) als closure dem User-Objekt zuordnen - das könnte klappen	Gregor Gabriel	2020-07-27T09:01:07.762Z
added 1 commit b2d24a11 - added comment to clarify why users email address is set to old one and the set back to new one Compare with previous version	Gregor Gabriel	2020-07-27T12:19:01.442Z
Das mit der Methode routeNotificationForMail hat so nicht funktioniert. Daher habe ich einen Kommentar spendiert, der erklärt warum die E-Mail-Adresse de Users auf die alte Adresse zurückgesetzt wird und dann gleich wieder auf die neue Adresse gesetzt wird.	Gregor Gabriel	2020-07-27T12:21:23.539Z
resolved all discussions	Sascha Immig	2020-07-27T15:04:35.235Z
mentioned in commit 81fd495c502d93e7485aa96adf40ecd887875d58	Sascha Immig	2020-07-28T06:48:58.243Z
merged	Sascha Immig	2020-07-28T06:48:58.453Z

Merge Commits

Title	Id	Author	Created at	Message
+ email to user is send, when someone else changes his password or email	32ef8c1f902e5d7fbe3b93e9d69e1e7dfe2e17a9	TUV\gabrielg	2020-07-17T13:14:57.000Z	+ email to user is send, when someone else changes his password or email - at the moment email is only send to new email address
added comment to clarify why users email address is set to old one and the set back to new one	b2d24a11e9c188ae1ef81a17681d4458df3730b7	TUV\gabrielg	2020-07-27T12:18:51.000Z	added comment to clarify why users email address is set to old one and the set back to new one
+ now old email account is infomed about email change	bddbe067f3b302b8b5a581d27b7f574368bd8890	TUV\gabrielg	2020-07-17T13:35:41.000Z	+ now old email account is infomed about email change