Details des Tickets
Beschreibung,Kommentare und Anhänge
added 23 commits
- aeb65c25...cc594cc6 - 22 commits from branch
develop - e2be22ff - Merge branch 'develop' into feature/LL-1252-link-reminder-to-group
Compare with previous version |
Markus Gerlach |
2022-10-11T12:38:06.926Z |
| assigned to @gabrielg |
Sascha Immig |
2022-10-11T15:43:52.351Z |
added 1 commit
- aa4f6816 - LL-1252 simplifies the process for the user by receiving the password reset email directly
Compare with previous version |
Markus Gerlach |
2022-10-12T13:56:51.744Z |
added 1 commit
- 8afdb4fc - LL-1252 revert to previous version
Compare with previous version |
Markus Gerlach |
2022-10-12T15:14:03.588Z |
added 1 commit
- b005b85b - LL-1252 remove unnecessary empty row
Compare with previous version |
Markus Gerlach |
2022-10-12T15:34:17.940Z |
| `$groupWithoutReminderAssignments` ist eine Collection.
count darauf zu nutzen ist ok, aber `$groupWithoutReminderAssignments->isNotEmpty()` wäre sprechender.
Das muss jetzt nicht unbedingt geändert werden, aber so für die Zukunft ein Improvement. |
Gregor Gabriel |
2022-10-18T07:35:49.801Z |
| ? Hast du geändert, weil dieser Reminder ja intern auch für nicht obligatorische Trainings gesendet wird, denke ich.
? Sollten wir hier envtl. ein "IF" spendieren und den Text davon abhängig machen, ob das Training obligatorisch ist oder nicht ?
@immigs - Das müsstest fast du entscheiden. |
Gregor Gabriel |
2022-10-18T07:43:53.668Z |
| Das tut so, da das Feld aber ein Boolean ist wäre `->default(true)` logischer.
Musst du jetzt aber nicht ändern, nur für's nächste mal. |
Gregor Gabriel |
2022-10-18T07:46:21.724Z |
| ? Frage wie oben, brauchen wir 2 Texte 1x obligatorisch und 1x optional
@immigs: hier bist du gefragt |
Gregor Gabriel |
2022-10-18T08:54:24.691Z |
| aktuell sind alle Trainings mit einem Enddatum eigentlich Pflichttrainings. In so fern, ist das inhaltlich nicht falsch.
Aber sehe keinen akuten Grund für diese Anpassnug. :) |
Sascha Immig |
2022-10-19T10:49:28.030Z |
| Doch, bitte anpassen. Und die Einrückung in den Zeilen 128 bis 136 korrigieren - 2 Tabs, nicht bis zum Pfeil von $user bitte. |
Sascha Immig |
2022-10-19T10:49:38.723Z |
| Bitte anpassen - das findet sonst keiner wieder... |
Sascha Immig |
2022-10-19T10:50:10.946Z |
| durch die Gruppeneinstellung kann ein Training jetzt ja eben doch optional zugeordnet sein |
Gregor Gabriel |
2022-10-19T10:53:24.686Z |
| wenn ich ein Training optional über eine Gruppe erhalte, so wäre das Wort "optional" für mich durchaus nicht uninteressant |
Gregor Gabriel |
2022-10-19T10:55:07.459Z |
| Sehe die Thematik.
Optional ist in diesem Kontext relativ. Das Training wird weiter in Reports etc. als verpflichtend gewertet. Es wird lediglich keine Erinnerung per Mail versendet...
Ulli spricht in diesem Kontext auch immer von den optionalen Trainings - m.E. gehört zu den optionalen Trainings deutlich mehr. Das Wort "optional" in diesem Kontext wift wieder Fragen auf, warum es dann als nicht erledigt in den Reports auftaucht etc. |
Sascha Immig |
2022-10-20T09:49:24.321Z |
| s.o.
~~auf jeden Fall brauchen wir zum gegenwärtigen Zeitpunkt keine 2 Texte.~~ |
Sascha Immig |
2022-10-20T09:50:47.272Z |
| Für jetzt gerne beide Texte (mit "verpflichtend" und ohne) einbauen. Bitte auf "optional" im Wording verzichten.
Textanpassungen müssen wir später nochmal angehen, wenn "echte" optionale Trainings kommen. |
Sascha Immig |
2022-10-20T09:53:24.917Z |
| Für jetzt gerne beide Texte (mit "verpflichtend" und ohne) einbauen. Bitte auf "optional" im Wording verzichten.
Textanpassungen müssen wir später nochmal angehen, wenn "echte" optionale Trainings kommen. |
Sascha Immig |
2022-10-20T09:53:46.589Z |
| changed this line in version 6 of the diff |
Markus Gerlach |
2022-11-03T13:38:06.179Z |
| changed this line in version 6 of the diff |
Markus Gerlach |
2022-11-03T13:38:06.546Z |
| changed this line in version 6 of the diff |
Markus Gerlach |
2022-11-03T13:38:07.003Z |
added 1 commit
- 92b49e88 - LL-1252 optimization of the previous solution
Compare with previous version |
Markus Gerlach |
2022-11-03T13:38:07.234Z |
| resolved all discussions |
Markus Gerlach |
2022-11-03T13:40:35.639Z |
| resolved all discussions |
Markus Gerlach |
2022-11-04T07:42:59.188Z |
added 1 commit
- c9427511 - composer update dompdf/dompdf -> 2.0.1
Compare with previous version |
Markus Gerlach |
2022-11-04T07:42:59.550Z |
| resolved all discussions |
Markus Gerlach |
2022-11-07T08:21:35.492Z |
added 112 commits
- c9427511...b1d9ce18 - 111 commits from branch
develop - 8ff37d22 - Merge branch 'develop' into feature/LL-1252-link-reminder-to-group
Compare with previous version |
Markus Gerlach |
2022-11-07T08:21:36.797Z |
| resolved all discussions |
Sascha Immig |
2022-11-08T10:50:12.159Z |
added 1 commit
- 829be9a1 - optimitzed annotation
Compare with previous version |
Sascha Immig |
2022-11-08T10:50:12.352Z |
| enabled an automatic merge when the pipeline for 829be9a1e1b33f09828c6d7681fb3f9ed5c5379b succeeds |
Sascha Immig |
2022-11-08T10:50:20.835Z |
| merged |
Sascha Immig |
2022-11-08T11:01:30.067Z |
| mentioned in commit 76fc15e858406ee70ef5ec63886ae906a5e0e7ef |
Sascha Immig |
2022-11-08T11:01:30.253Z |
| Title |
Id |
Author |
Created at |
Message |
| optimitzed annotation |
829be9a1e1b33f09828c6d7681fb3f9ed5c5379b |
TUV\immigs |
2022-11-08T10:50:04.000Z |
optimitzed annotation
|
| LL-1252 revert to previous version |
8afdb4fcec4dafc80d622bc9a50f4fc2eadb19be |
markusgerlach |
2022-10-12T15:14:00.000Z |
LL-1252 revert to previous version
|
| Merge branch 'develop' into feature/LL-1252-link-reminder-to-group |
8ff37d228df03df596aad7d8f4b1868bc61c8974 |
gerlachm |
2022-11-07T08:21:28.000Z |
Merge branch 'develop' into feature/LL-1252-link-reminder-to-group
# Conflicts:
# composer.lock
|
| LL-1252 optimization of the previous solution |
92b49e880bdb866eb1b4d8a8e5c159f7e2297c7f |
gerlachm |
2022-11-03T13:38:01.000Z |
LL-1252 optimization of the previous solution
|
| LL-1252 simplifies the process for the user by receiving the password reset email directly |
aa4f6816aa2b05b7a4a844c73b0d736d50a3c496 |
markusgerlach |
2022-10-12T13:56:46.000Z |
LL-1252 simplifies the process for the user by receiving the password reset email directly
Set directly the email rerquest value from url to password/reset form
|
| LL-1252 groups with disabled reminders will not receive email notifications.… |
aeb65c25226411ef9e8f267fbc68e4fa64fab21c |
markusgerlach |
2022-10-11T09:43:27.000Z |
LL-1252 groups with disabled reminders will not receive email notifications. database notifications are still transmitted. Adjustment of the database notification.
|
| LL-1252 remove unnecessary empty row |
b005b85b1564cf4a002cc25bd4134e2047bfa72b |
markusgerlach |
2022-10-12T15:34:14.000Z |
LL-1252 remove unnecessary empty row
|
| composer update dompdf/dompdf -> 2.0.1 |
c9427511073af3933a174b4500e10793f587c970 |
gerlachm |
2022-11-04T07:42:55.000Z |
composer update dompdf/dompdf -> 2.0.1
|
| Merge branch 'develop' into feature/LL-1252-link-reminder-to-group |
e2be22ff379c7b39795f475a780d4f320893bbe0 |
markusgerlach |
2022-10-11T12:38:03.000Z |
Merge branch 'develop' into feature/LL-1252-link-reminder-to-group
# Conflicts:
# app/Console/Commands/SendTrainingReminders.php
|