Details des Tickets

Beschreibung,Kommentare und Anhänge

merge request id 149
TitelFeature/ll 186 refactor training progress
Beschreibung**LL-186 Umbau des Trainingsfortschritts** Der Trainingsfortschritt darf nicht mehr an der Kombination aus User und Training gespeichert werden, sondern muss explizit an der entsprechenden Trainingszuweisung gebucht werden. Dies betrifft sowohl den Fortschritt des gesamten Trainings wie den Fortschritt der einzelnen Items. Beim speichern der Sessions und der damit verbundenen Aktualisierung der Fortschritte, muss sichergestellt sein, dass nur aktive Buchungen aktualisiert werden. http://jira.learningsystem.de/browse/LL-186 Gregor Gabriel hat einen Kommentar hinzugefügt - 12/Feb/20 9:02 - bearbeitet Zunächt werten TrainingProgress und TrainingItemProgress mit dem Attribut training_assignment_id versehen. Das schein zu reichen um eine neue Lernsession zu starten und alle passenden TrainingItemProgress Objekte zu aktualisieren. ABER wenn man weiterlernen will, wie stellt man da sicher, dass der bereits vorhandene Lernfortschritt aus dem Zeitraum der Freischaltung ist? Wir prüfen hier ob eine Lern-Session existiert, die in den Freischaltungszeitraum fällt. Existieren mehrere Freischaltungen - eine in deren Zeitraum die letzte Lern-Session nicht fällt und eine in deren Zeitraum die letzte Lern-Session fällt - dann wird die Lern-Session fortgesetzt und gilt für beide Freischaltungen. Bei Freischaltung-Überlappungen zeigen wir immer den besten Lernfortschritt. - Es kann aber passieren, dass bei überlappenden Freischaltungen, ein Lerner heute ins System geht und feststellt, dass er nichts zu tun hat. Würde er dann das Training nur kurz starten, ohne es durchzuarbeiten, so würde er den Lernstand in die neue Freischaltung dauerhaft übernehmen. Tut er es nicht, so muss er das Training an nächsten Morgen komplett absolvieren, wenn die ältere Freischaltung dann abgelaufen ist. Gregor Gabriel hat einen Kommentar hinzugefügt - Gestern 9:05 Wir zeigen ab jetzt bei den Trainings unterhalb eines Users die Buchungen an, so dass Trainings doppelt erscheinen können
Statusmerged
Created at 2020-02-14T10:04:53.227Z
Updated at 2020-02-25T11:42:46.254Z
Source branchfeature/LL-186-refactor-training-progress
Target branchdevelop
AuthorGregor Gabriel
AssigneeMarcel Pociot

Discussion notes

added 1 commit
  • a22a8865 - Update MyTrainingController.php
Compare with previous version
Marcel Pociot 2020-02-21T13:04:34.342Z
Das verursacht in der Form leider Probleme, bei folgendem Code: ``` $training = Training::find(1); $latestAssignment = $training->getLatestTrainingAssignment(); $latestAssignmentForOtherUser = $training->getLatestTrainingAssignment($differentUser); ``` In dem Fall sind beide zurückgegebenen Variablen gleich, weil sie in dem Trainings-Model zwischengespeichert werden. Hier macht es vielleicht mehr Sinn, sich das Training Assignment für die user-id zu merken. Marcel Pociot 2020-02-25T09:35:06.934Z
Macht Sinn - baue ich um Gregor Gabriel 2020-02-25T09:47:36.440Z
changed this line in version 3 of the diff Gregor Gabriel 2020-02-25T10:22:54.210Z
resolved all discussions Gregor Gabriel 2020-02-25T10:22:54.272Z
added 1 commit
  • 7560508f - + fixed problem reportes by Marcel - method…
Compare with previous version
Gregor Gabriel 2020-02-25T10:22:54.450Z
Hab's korrigiert - pipeline läuft Gregor Gabriel 2020-02-25T10:23:35.051Z
resolved all discussions Marcel Pociot 2020-02-25T11:42:24.751Z
merged Marcel Pociot 2020-02-25T11:42:46.307Z
mentioned in commit 9e5cf4990a5c002065d8cb2e606ca08da2877404 Marcel Pociot 2020-02-25T11:42:46.470Z

Merge Commits

Title Id Author Created at Message
Initial impementation without any testing 147bed40d3051efd0aae9496eb62f16086c25629 TUV\gabrielg 2020-02-12T15:56:28.000Z Initial impementation without any testing
+ all links to certificates now are based on TrainingAssignment 1b19b003747ea41075189bdc4c5d714f95ad9efc TUV\gabrielg 2020-02-14T08:38:42.000Z + all links to certificates now are based on TrainingAssignment
+ fixed test errors 2a457ecc6a3178ff455933ce8a8949a2de8443da TUV\gabrielg 2020-02-13T10:46:07.000Z + fixed test errors
+ added TrainingAssignment to certificate preview code 320378fbd2cfa84e7875d280a8988cf04ab6a02d TUV\gabrielg 2020-02-13T14:20:43.000Z + added TrainingAssignment to certificate preview code
+ training listing in a users administration data sheet now is based on all training assignments 6e1aa9dacf8480f863533cd345f4a62b46c71689 TUV\gabrielg 2020-02-13T09:11:13.000Z + training listing in a users administration data sheet now is based on all training assignments + fixed tests
+ listing certified trainings in profile now by TrainingAssignment 6e4fbf9c2ad550c235d94cdf0773b4e0e6a998be TUV\gabrielg 2020-02-13T14:18:58.000Z + listing certified trainings in profile now by TrainingAssignment
+ fixed problem reportes by Marcel - method… 7560508fc592cccff0b99ea366fa63a9ea2efac2 TUV\gabrielg 2020-02-25T10:22:33.000Z + fixed problem reportes by Marcel - method Training::getLatestTrainingAssignment now works correctly when invoked for lots of differen users
Update MyTrainingController.php a22a88655a11dfc692c51b257c08391713c83cb1 Marcel Pociot 2020-02-21T13:04:28.000Z Update MyTrainingController.php
- renamed function userTrainingProgress to trainingProgress aac334d1a70e96627a5662f93897b1ebd06023be TUV\gabrielg 2020-02-14T06:37:06.000Z - renamed function userTrainingProgress to trainingProgress
+ added TrainingAssignment in TrainingProgress generation in TrainingProgressSeeder d23e6ce9018941c2882da5f5eb85c10322b20cf0 TUV\gabrielg 2020-02-13T14:15:31.000Z + added TrainingAssignment in TrainingProgress generation in TrainingProgressSeeder
+ generation function for certificates now uses TrainingAssignment instead of Training f07ae640ea369e23caeb3e6fb35f2ff37899ac98 TUV\gabrielg 2020-02-13T15:35:53.000Z + generation function for certificates now uses TrainingAssignment instead of Training
+ deletion of certificae file now uses TrainingAssignment instead of Training fd1e8dfca5a898dfc419684d832990c8398d8d6e TUV\gabrielg 2020-02-14T09:43:22.000Z + deletion of certificae file now uses TrainingAssignment instead of Training + build of internal certificate file name now is centralized in one method and file name additionally contains training_assignment_id