Details des Tickets

Beschreibung,Kommentare und Anhänge

merge request id 524
TitelFeature/ll 910 adaptation training queries number of new messages
Beschreibung
Statusmerged
Created at 2021-11-17T14:04:37.062Z
Updated at 2021-11-23T09:17:43.149Z
Source branchfeature/LL-910-adaptation-training-queries-number-of-new-messages
Target branchdevelop
AuthorMarkus Gerlach
AssigneeGregor Gabriel

Discussion notes

assigned to @gabrielg Sascha Immig 2021-11-22T07:31:38.410Z
es macht m.E. keinen Sinn, dass der close() Methode ein Training und eine Id einer correspondence übergeben werden musss. Die Id hat das Objekt ja selber und den Bezug zum Training ja auch. Gregor Gabriel 2021-11-22T08:53:46.341Z
Auch hier wie schon oben genannt. Die open-Methode der Klasse Correspondence sollte keine Parameter benötigen, da sie bereits alle Informationen besitzt. Gregor Gabriel 2021-11-22T08:55:49.678Z
s.o. Signatur sollte leere sein Gregor Gabriel 2021-11-22T09:41:10.297Z
s.o. Was bringt eigentlich die Aufsplittung in open() und openAsUser() ? Gregor Gabriel 2021-11-22T09:51:27.775Z
Signatur s.o. Gregor Gabriel 2021-11-22T09:52:03.454Z
Signatur s.o. Was bringt eigentlich die Aufsplittung in open() und openAsUser() ? Gregor Gabriel 2021-11-22T09:52:45.320Z
1. Das $training hängt doch an der $question schon dran. Sollte man daher hier nicht übergeben müssen. 2. Statt dem SenderName würde ich lieber den Sender mit übergeben und in der Notification dann den Sendername daraus bauen. Dann gibt es für diese Notification nur eine Stelle wo der Sendername gebaut wird. 3. $question sollte das Correspondance-Objekt sein Gregor Gabriel 2021-11-22T10:00:07.222Z
s.o. Gregor Gabriel 2021-11-22T10:01:08.860Z
changed this line in version 2 of the diff Markus Gerlach 2021-11-22T12:26:49.860Z
changed this line in version 2 of the diff Markus Gerlach 2021-11-22T12:26:50.174Z
changed this line in version 2 of the diff Markus Gerlach 2021-11-22T12:26:50.267Z
changed this line in version 2 of the diff Markus Gerlach 2021-11-22T12:26:50.368Z
changed this line in version 2 of the diff Markus Gerlach 2021-11-22T12:26:50.481Z
changed this line in version 2 of the diff Markus Gerlach 2021-11-22T12:26:50.603Z
changed this line in version 2 of the diff Markus Gerlach 2021-11-22T12:26:50.713Z
added 1 commit
  • bd3dd155 - LL-910-adaptation-training-queries-number-of-new-messages
Compare with previous version
Markus Gerlach 2021-11-22T12:26:50.833Z
Ja richtig. War so aufgebaut da hier als Basis die Funktion storeReply() verwendet wurde. Markus Gerlach 2021-11-22T12:36:00.292Z
ja richtig. War so aufgebaut da hier als Basis die Funktion storeReply() verwendet wurde. Markus Gerlach 2021-11-22T12:36:22.905Z
thx Markus Gerlach 2021-11-22T12:36:33.952Z
War so Aufgebaut da hier als Basis die Funktion storeAnswer() verwendet wurde. Hab's jetzt verinfacht. Markus Gerlach 2021-11-22T12:39:23.588Z
War so Aufgebaut da hier als Basis die Funktion storeAnswer() verwendet wurde. Hab's jetzt verinfacht. Markus Gerlach 2021-11-22T12:39:44.917Z
thx Markus Gerlach 2021-11-22T12:39:51.871Z
thx Markus Gerlach 2021-11-22T12:40:21.208Z
alles gut Gregor Gabriel 2021-11-22T14:44:37.587Z
added 1 commit
  • 60d6e93c - + fixed probelem occured on function signature change
Compare with previous version
Gregor Gabriel 2021-11-22T16:05:26.381Z
ich glaube die ID war storeReply() dran, weil dort die ID der Message, zu der geantwortet wird, übergeben wird. Sascha Immig 2021-11-22T19:52:33.896Z
ich glaube die ID war storeAnswer() dran, weil dort die ID der Message, zu der geantwortet wird, übergeben wird. Sascha Immig 2021-11-22T19:52:52.749Z
Passt alles Markus Gerlach 2021-11-23T07:54:35.330Z
resolved all discussions Gregor Gabriel 2021-11-23T09:17:32.132Z
merged Gregor Gabriel 2021-11-23T09:17:43.269Z
mentioned in commit dbca328947a1408830d81a67e5f60db15b53728f Gregor Gabriel 2021-11-23T09:17:43.724Z

Merge Commits

Title Id Author Created at Message
+ fixed probelem occured on function signature change 60d6e93c9621ef68706c82c78d142bfc6d6067ce TUV\gabrielg 2021-11-22T16:05:17.000Z + fixed probelem occured on function signature change + fixed that closed questions did not count as answered
Merge branch 'develop' into feature/LL-910-adaptation-training-queries-number-of-new-messages 813dcc8e19e730fad4a076f8f08bb78c4ecdd0ee markusgerlach 2021-11-17T13:35:09.000Z Merge branch 'develop' into feature/LL-910-adaptation-training-queries-number-of-new-messages
LL-910-adaptation-training-queries-number-of-new-messages bd3dd155d2a15971d2977018d1b4dc008d6b2710 markusgerlach 2021-11-22T12:26:46.000Z LL-910-adaptation-training-queries-number-of-new-messages
LL-910-adaptation-training-queries-number-of-new-messages c2c390ae65bf8a864bd6b2fd1822986424321443 markusgerlach 2021-11-17T10:37:23.000Z LL-910-adaptation-training-queries-number-of-new-messages