Details des Tickets
Beschreibung,Kommentare und Anhänge
| Statt default(1), sollte der nur für alle Superadmins auf 1 stehen... Macht keinen Unterschied in der aktuellen Konstellation, aber es wäre sauberer von der Datenhaltung.
Daher bitte default(0) setzen und alle aktuellen Superadmins nachträglich auf 1 Setzen. |
Sascha Immig |
2021-08-26T08:07:29.815Z |
| ich glaube, das 2. Parameter (die 1) ist der Default-Wert... Bitte kurz verifizieren und dann auf 0 setzen, damit es mit der Datenbank übereinstimmt. |
Sascha Immig |
2021-08-26T08:12:25.656Z |
| Hier wird es was komplizierter:
Ja, korrekt umgesetzt ist, dass die Box nur den Superadmins zur Verfügung steht. Ich hätte die Box aber gerne auch nur dann angezeigt, wenn der zu bearbeitende User ebenfalls ein Superadmin ist.
Bitte kurzes jQuery Skript, das den Block (ID vergeben) abhängig von der Checkbox "Superadmin" ein und ausblendet.
(Wenn ich das richtig sehe, wird beim Speichern schon berücksitigt, dass mail_notifications explizit auf false gesetzt wird, wenn der Wert nicht mitgesendet wird. Bitte kurz verifizieren, dass der Wert beim Ausblenden nicht gesendet wird (sonst könnte man anhaken und danach Superadmin entfernen und es würde fälschicher weise gespeichert, obwohl die Box nicht angezeigt wird).
Wenn es mitgesendet wird, beim ausblenden der Blocks mail_notification einfach im jQuery auf false setzen... |
Sascha Immig |
2021-08-26T08:14:58.219Z |
| wie per skype besprochen...
wird auf 0 gesetzt. Superadmins manuell... |
Markus Gerlach |
2021-08-26T08:29:41.889Z |
| erledigt |
Markus Gerlach |
2021-08-26T10:11:40.923Z |
| echo Form::checkbox('name', 'value', true);
https://laravel.com/docs/4.2/html
1 (Rückgabewert) muss stehen bleiben sonst funktioniert es nicht |
Markus Gerlach |
2021-08-26T10:30:21.275Z |
| changed this line in version 2 of the diff |
Markus Gerlach |
2021-08-26T10:32:38.880Z |
| changed this line in version 2 of the diff |
Markus Gerlach |
2021-08-26T10:32:39.542Z |
added 1 commit
- 25b494f5 - LL-954-notification-of-superadmins-via-flag
Compare with previous version |
Markus Gerlach |
2021-08-26T10:32:39.826Z |
| resolved all discussions |
Sascha Immig |
2021-08-26T14:51:57.395Z |
| merged |
Sascha Immig |
2021-08-27T07:05:34.814Z |
| mentioned in commit 913e5bfbc81c3a36beaae8e4329edfbe3940417e |
Sascha Immig |
2021-08-27T07:05:34.909Z |