Details des Tickets
Beschreibung,Kommentare und Anhänge
| Bitte nicht die ID `'1'` direkt verwenden.
besser
> $role = Role::where(['name' => User::ROLE_CLIENT_ADMIN, 'default' => true])->first();
und dann $role->id nutzen |
Gregor Gabriel |
2022-03-07T10:41:45.675Z |
| @immigs
Ich weiss, Sacha hat in der Besprechung gesagt, wir machen das im Benutzer-Datenblatt.
Aber eigentlich gehört das ins Profil und wird nur Mandanten-Admins angezeigt, oder?
Lass uns hierzu mal noch bei Sascha fragen, ob er es nicht doch im Profil will. |
Gregor Gabriel |
2022-03-07T12:28:58.534Z |
| Im Profil ändert jeder User seine Daten. Im Benutzerdatenblatt kann der Client-Admin Einstellungen vornehmen. Die Abschaltung der Mailbenachrichtigung gehört prinzipiell zu letzterem. Daher sollte das im Benutzerdatenblatt verfügbar sein.
Zudem wollte ich eine Option ohne großen Eingriff. Da die Checkbox bereits vorhanden ist, kam mir die Stelle günstig vor.
Dennoch gefällt mir die Idee, es zusätzlich im Profil anzuzeigen. Dort darf die Option aber definitiv nur verfügbar sein, wenn man selber Client-Admin ist. Ich würde das als separaten Task aufnehmen - wenn ich meine Notizen grade richtig im Kopf hab, wird es kurzfristig weitere Einstellungs-Optionen geben, die dann Berücksichtigung im Profil finden könnten. Danke für die Anregung! :) |
Sascha Immig |
2022-03-07T12:43:53.224Z |
| Task wurde erstellt (LL-1129). Ich würde noch sammeln, was sonst noch ins Profil sollte. |
Sascha Immig |
2022-03-07T13:13:01.793Z |
added 1 commit
- adc022a7 - LL-1119-mail-checkbox-also-for-client-admins
Compare with previous version |
Markus Gerlach |
2022-03-07T13:33:29.446Z |
| resolved all discussions |
Markus Gerlach |
2022-03-07T13:45:54.811Z |
| resolved all discussions |
Gregor Gabriel |
2022-03-07T16:11:59.013Z |
added 2 commits
- 24182e96 - + added additional info at checkbox which allows to skip email on user generation
- 39c303f8 - Merge branch 'feature/LL-1119-mail-checkbox-also-for-client-admins' of…
Compare with previous version |
Gregor Gabriel |
2022-03-07T16:11:59.252Z |
| changed this line in version 4 of the diff |
Gregor Gabriel |
2022-03-08T15:23:53.669Z |
| resolved all discussions |
Gregor Gabriel |
2022-03-08T15:23:53.889Z |
added 1 commit
- f337db21 - [ci-skip] added comments
Compare with previous version |
Gregor Gabriel |
2022-03-08T15:23:54.055Z |
| merged |
Gregor Gabriel |
2022-03-09T07:17:13.828Z |
| mentioned in commit 6fa76387c44d2f74d784dc592223e30e1283c95d |
Gregor Gabriel |
2022-03-09T07:17:13.888Z |
| Title |
Id |
Author |
Created at |
Message |
| + added additional info at checkbox which allows to skip email on user generation |
24182e9643179e86c97f73e65b09e872f34d4643 |
TUV\gabrielg |
2022-03-07T16:08:43.000Z |
+ added additional info at checkbox which allows to skip email on user generation
+ fixed wrong recipient for ClientVoucherDiscountEndedToday mail
+ wrapped test if user wants mail notifications in a scope un user class
- some notifications / mails still need to be restrict to a positive users.mail_notifications attribute
|
| Merge branch 'feature/LL-1119-mail-checkbox-also-for-client-admins' of… |
39c303f8f38610dbdb798c1157b4579f8eeaa10a |
TUV\gabrielg |
2022-03-07T16:11:41.000Z |
Merge branch 'feature/LL-1119-mail-checkbox-also-for-client-admins' of https://gitlab.learningsystem.de/lms/seventeen into feature/LL-1119-mail-checkbox-also-for-client-admins
|
| LL-1119-mail-checkbox-also-for-client-admins |
adc022a704b6d6cd7f0ce361b4937deebee399fa |
markusgerlach |
2022-03-07T13:33:21.000Z |
LL-1119-mail-checkbox-also-for-client-admins
|
| [ci-skip] added comments |
f337db21d8cceba5740beb7b4798db03a612b6ef |
TUV\gabrielg |
2022-03-08T15:23:33.000Z |
[ci-skip] added comments
|
| LL-1119-mail-checkbox-also-for-client-admins |
f45ad5fea20cdb8f706bc678f261ec3905f84044 |
markusgerlach |
2022-03-04T12:26:02.000Z |
LL-1119-mail-checkbox-also-for-client-admins
|