Details des Tickets

Beschreibung,Kommentare und Anhänge

merge request id 212
TitelFeature/ll 329 notify new employees
BeschreibungBenachrichtigung über angelegte Benutzer (http://jira.learningsystem.de/browse/LL-329) Szenario 1: Wird ein neuer Nutzer von einem Super-Admin angelegt, wird der neu angelegte Nutzer - sofern er eine Mailadresse hat - über seinen Account informiert. Zusätzlich geht eine Mail / Notification an den / die Client-Admin(s). Szenario 2: Wird ein neuer Nutzer vom Client-Admin oder einem andern Nutzer mit entsprechendem Recht (per Rolle oder direkt) angelegt, wird der neu angelegte Nutzer - sofern er eine Mailadresse hat - über seinen Account informiert. Zusätzlich geht eine Mail / Notification an alle (anderen) Client-Admins dieses Mandanten, die über den neuen Nutzer informiert. Ausgenommen von dieser Mail ist der ausführende Client-Admin. Szenario 3: Wird ein Benutzer-Import ausgeführt (das mach nie der Superadmin) so werden die angelegten User nur dann informiert, wenn es in der Import-Datei ausdrücklich gewünscht wurde oder umgekehrt. Eine Benachrichtigung aller Mandanten-Admins unterbleibt.
Statusmerged
Created at 2020-07-01T14:42:05.907Z
Updated at 2020-07-09T11:56:24.870Z
Source branchfeature/LL-329-notify-new-employees
Target branchdevelop
AuthorGregor Gabriel
AssigneeSascha Immig

Discussion notes

@pociotm Bei meinen Test ist der Code rein. Eine E-Mail wurde aber nicht im Laravel-Log eingetragen. Beim Anlegen eines Users wird die E-Mail ins Log geschrieben. ??? Ist das Korrekt, dass bei der Ausführung von Jobs versendete E-Mails nicht im Laravel-Log angezeigt werden ??? Gregor Gabriel 2020-07-01T14:46:29.357Z
changed this line in version 2 of the diff Marcel Pociot 2020-07-08T08:59:48.601Z
resolved all discussions Marcel Pociot 2020-07-08T08:59:48.909Z
added 1 commit
  • 0d936274 - Add notification test and fix typo in class name
Compare with previous version
Marcel Pociot 2020-07-08T08:59:49.245Z
Anmerkung zur Übersetzung: das Onboarding-Modal nutzt für "Unterweisungslösung" das englische "entrepeneural training solution". Hier habe ich "Instructional System" gesehen... wir sollten uns da auf eines einigen und das global verwenden. "... Training Solution" finde ich persönlich netter als etwas mit "instruction"... Sascha Immig 2020-07-08T18:53:06.296Z
added 1 commit
  • 29faaceb - Mark users as active during import
Compare with previous version
Marcel Pociot 2020-07-09T05:55:41.118Z
added 1 commit
  • a7de5abb - "instructional system" => "entrepeneural training solution"
Compare with previous version
Gregor Gabriel 2020-07-09T08:52:39.186Z
merged Sascha Immig 2020-07-09T11:55:29.648Z
mentioned in commit dde16e1a127072b6b5be4aebfff16be2e822bd6f Sascha Immig 2020-07-09T11:55:30.034Z
changed the description Sascha Immig 2020-07-09T11:56:24.934Z

Merge Commits

Title Id Author Created at Message
Add notification test and fix typo in class name 0d936274d03271c2db1b26d6d3e3e7c4ae8b5611 Marcel Pociot 2020-07-08T08:59:39.000Z Add notification test and fix typo in class name
WIP 19d971ba4ca4635c6514bc758d405c81bb9a7ba9 TUV\gabrielg 2020-06-30T14:54:02.000Z WIP sending mail to user when accout is created in administration or optional in user import
Mark users as active during import 29faaceb7104d43ab0605b3e7813b00f7c9e3374 Marcel Pociot 2020-07-09T05:55:34.000Z Mark users as active during import
+ sending mail and database message to all client admins 84fe5e9bcbe44a85d0c18afdbc9a22c8c4cbf6ee TUV\gabrielg 2020-07-01T10:32:18.000Z + sending mail and database message to all client admins + small fixes
+ user notification about regitered account works now 8f326214edbbb6223c18c8601c896626b0252c26 TUV\gabrielg 2020-07-01T08:08:47.000Z + user notification about regitered account works now + added as well to user import, but was not able ti verify it, as message is not logged to laravel log on import ???
"instructional system" => "entrepeneural training solution" a7de5abb672a35905272ef664204ffd5e8346a3c TUV\gabrielg 2020-07-09T08:52:17.000Z "instructional system" => "entrepeneural training solution"