Last notification date for user qualifications seem to be wrong #159

Closed
opened 2024-04-19 16:47:25 +02:00 by jost · 7 comments
jost commented 2024-04-19 16:47:25 +02:00 (Migrated from gitlab.uniworx.de)

Die Spalte "Letzte Benachrichtigung" der Tabelle der individuellen Qualifikation stimmt nicht überein mit der Spalte "LMS Versand Benachrichtigung" unter LMS/'E-Learning'

Die Spalte **"Letzte Benachrichtigung"** der Tabelle der individuellen Qualifikation stimmt nicht überein mit der Spalte **"LMS Versand Benachrichtigung"** unter LMS/'E-Learning'
jost commented 2024-04-19 16:47:25 +02:00 (Migrated from gitlab.uniworx.de)

assigned to @jost

assigned to @jost
jost commented 2024-04-19 17:02:12 +02:00 (Migrated from gitlab.uniworx.de)

Erstes Prüfungsergebnis

Die Spalte bezieht sich nur auf solche Benachrichtigungen, die bei Änderung der Gültigkeit der Qualifikation versendet werden, also z.B. bei Entzug der Fahrlizenz.

Wird ein E-Learning gestartet und rechtzeitig erfolgreich beendet, so bleibt der Wert unverändert.
Dies ist technische notwendig, damit mehrere Instanzen von FRADrive oder nach einem Neustart nicht mehrere Benachrichtigungen zum gleichen Ereignis versendet werden.

Vorschläge zu Lösung:

  1. Name der Spalte ändern in "Letzte Gültigkeitsänderung" oder so ähnlich, mit einem erklärenden Tooltip?
  2. Spalte nicht mehr anziegen
  3. In der DB eine weitere Spalte einführen, welche jegliche Benachrichtigungen im Zusammenhang mit dieser Qualifikation berücksichtigt und dann diese Spalte stattdessen anzeigen (wäre natürlich aufwändiger).

@wiese Welche Option sollte es sein?

## Erstes Prüfungsergebnis Die Spalte bezieht sich nur auf solche Benachrichtigungen, die bei Änderung der Gültigkeit der Qualifikation versendet werden, also z.B. bei Entzug der Fahrlizenz. Wird ein E-Learning gestartet und rechtzeitig erfolgreich beendet, so bleibt der Wert unverändert. Dies ist technische notwendig, damit mehrere Instanzen von FRADrive oder nach einem Neustart nicht mehrere Benachrichtigungen zum gleichen Ereignis versendet werden. ### Vorschläge zu Lösung: 1. Name der Spalte ändern in "Letzte Gültigkeitsänderung" oder so ähnlich, mit einem erklärenden Tooltip? 2. Spalte nicht mehr anziegen 3. In der DB eine weitere Spalte einführen, welche jegliche Benachrichtigungen im Zusammenhang mit dieser Qualifikation berücksichtigt und dann diese Spalte stattdessen anzeigen *(wäre natürlich aufwändiger)*. @wiese Welche Option sollte es sein?
jost commented 2024-04-19 17:02:25 +02:00 (Migrated from gitlab.uniworx.de)

assigned to @wiese and unassigned @jost

assigned to @wiese and unassigned @jost
wiese commented 2024-04-22 09:26:05 +02:00 (Migrated from gitlab.uniworx.de)

Ich würde es nicht mehr anzeigen. Das ist wahrscheinlich das einfachste. Der neue Name wäre auch nicht richtig, da bei einer Verlängerung durch das E-Learning, das Datum auch nicht geändert wird. Aufwendigere Anpassungen würde ich erst vornehmen, wenn Phase A angeschlossen ist. Kannst es auch erstmal so lassen. Ist ja nur mir aufgefallen. ich wollte es nur verstehen...

Ich würde es nicht mehr anzeigen. Das ist wahrscheinlich das einfachste. Der neue Name wäre auch nicht richtig, da bei einer Verlängerung durch das E-Learning, das Datum auch nicht geändert wird. Aufwendigere Anpassungen würde ich erst vornehmen, wenn Phase A angeschlossen ist. Kannst es auch erstmal so lassen. Ist ja nur mir aufgefallen. ich wollte es nur verstehen...
jost commented 2024-04-22 11:31:44 +02:00 (Migrated from gitlab.uniworx.de)

Naja, nach meiner Lesart wäre eine "Verlängerung" keine "Änderung der Gültigkeit", da die Lizenz ja durchweg gültig blieb.

Ich habe mal den Namen geändert, einen Tooltip hinzugefügt und die Spalte trotzdem entfernt (also nur Auskommentiert, da es vielleicht doch hilfreich sein könnte zu sehen, ob jemand über einen Entzug benachrichtigt wurde oder nicht - aber andererseits kann man ja im PrintCenter zumindest alle gedruckten Nachrichten sehen. Eventuell kann man darüber nachdenken, eine Benutzer-Nachrichtenseite zu bauen, die alle Briefe und Emails zu dem Nutzer anzeigt - vorhanden sind die Daten ja, man müsste nur die Anzeige bauen. @wiese Wenn sinnvoll, dann ein Issue mit Link zu diesem anlegen und Prio/Milestone entsprechend einstellen.

Naja, nach meiner Lesart wäre eine "Verlängerung" keine "Änderung der Gültigkeit", da die Lizenz ja durchweg gültig blieb. Ich habe mal den Namen geändert, einen Tooltip hinzugefügt und die Spalte trotzdem entfernt (also nur Auskommentiert, da es vielleicht doch hilfreich sein könnte zu sehen, ob jemand über einen Entzug benachrichtigt wurde oder nicht - aber andererseits kann man ja im PrintCenter zumindest alle gedruckten Nachrichten sehen. Eventuell kann man darüber nachdenken, eine Benutzer-Nachrichtenseite zu bauen, die alle Briefe und Emails zu dem Nutzer anzeigt - vorhanden sind die Daten ja, man müsste nur die Anzeige bauen. @wiese Wenn sinnvoll, dann ein Issue mit Link zu diesem anlegen und Prio/Milestone entsprechend einstellen.
jost (Migrated from gitlab.uniworx.de) closed this issue 2024-04-22 11:50:22 +02:00
wiese commented 2024-04-22 19:01:30 +02:00 (Migrated from gitlab.uniworx.de)

mentioned in issue #160

mentioned in issue #160
jost commented 2024-06-19 17:53:20 +02:00 (Migrated from gitlab.uniworx.de)

mentioned in commit fd6a5384d3

mentioned in commit fd6a5384d3517958a3c7726e32eed3bad197a591
This repo is archived. You cannot comment on issues.
No Milestone
No project
No Assignees
1 Participants
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: fraport/fradrive-old#159
No description provided.