Details des Tickets

Beschreibung,Kommentare und Anhänge

merge request id 500
TitelLL-994-user-export-department-and-position-missing
Beschreibung
Statusmerged
Created at 2021-10-06T09:44:45.391Z
Updated at 2021-10-11T07:33:39.438Z
Source branchfeature/LL-994-user-export-department-and-position-missing
Target branchdevelop
AuthorMarkus Gerlach
AssigneeSascha Immig

Discussion notes

Bitte nochmals die Abteilung verifizieren: Mein Abteilungsbaum sieht wie folgt aus (ohne Root): * Buchhaltung * Lohnbuchhaltung * Fertigung * Materialforschung meine User sind wie folgt zugeordnet: * P. Dolittle -> ACE Chemicals | Fertigung (Mitarbeiter) * Frank N. Further -> ACE Chemicals | ... | Materialforschung (Mitarbeiter) * Harleen Quinzel -> ACE Chemicals | Buchhaltung (Leitung) * Frank N. Stein -> ACE Chemicals | Fertigung (Leitung) im Excel habe ich: ``` | User | Abteilung | | --------- | ----------------- | | Quinzel | Buchhaltung | | Stein | Fertigung | | Doolittle | Fertigung | | Further | Materialforschung | ``` Müsste bei Frank N. Further nicht `Fertigung | Materialforschung` im Excel ausgegeben werden? Lt. Code wird im `implode()` auch die Pipe genutzt. Scheinbar ist `$user->organizationUnits` aber nur der letzte Knoten und nicht der komplette Ast. Sascha Immig 2021-10-07T11:24:48.275Z
@gabrielg ' User über mehrere Zeilen einfügen. Falls der Mitarbeiter mehreren Abteilung zugerdnet ist: Jede Abteilung in einzelner Zeile einfügen. ' Ich werde dies entsprechend anpassen. Markus Gerlach 2021-10-07T13:19:47.238Z
Wichtig ist, dass es Export auf den Import passt. Würde ich die Datei so importieren, würde Frank N. Further m.E. nicht korrekt einsortiert. Sascha Immig 2021-10-07T13:31:47.730Z
Dieser Kommentar kann erstmal ignoriert werden. Ich bin davon ausgegangen das | für die Trennung einzelner Abteilungen gilt und nicht für eine Pfadstruktur... @immigs sascha 'Müsste bei Frank N. Further nicht' Ja ist natürlich Auslegungssache er ist dieser ja nicht direkt zugeordnet. Ich hätte diese erwartet wie du es in der Excel siehst. Die Frage wäre wenn auch ,`Fertigung` dazu kommen soll, wie bildet man es ab? Auch im Hinblick auf mögliche Erweiterung des bisherigen imports. Laut Erläuterung, auf der impport Seite, wird hier nur eine Abteilung unterstützt. Wenn die Ebenen davor mit rein sollen dann würde sowas in die Richtung empfehlen Fertigung-->Materialforschung Die Frage wäre dann noch alle unter Ebenen rein? Root-->Fertigung-->Materialforschung Markus Gerlach 2021-10-07T13:33:08.700Z
added 1 commit
  • d7550486 - LL-994-user-export-department-and-position-missing
Compare with previous version
Markus Gerlach 2021-10-08T10:57:51.956Z
resolved all discussions Sascha Immig 2021-10-11T07:33:17.667Z
merged Sascha Immig 2021-10-11T07:33:39.606Z
mentioned in commit 232c98baddebf0567cb26d16e59b08a8db24db57 Sascha Immig 2021-10-11T07:33:40.136Z

Merge Commits

Title Id Author Created at Message
LL-994-user-export-department-and-position-missing 2c5869271b3ec4cf790c66c3765e7efc8a2dfaa1 markusgerlach 2021-10-06T09:30:38.000Z LL-994-user-export-department-and-position-missing
LL-994-user-export-department-and-position-missing d75504863dde295e37da44d69e5895f7ba746352 markusgerlach 2021-10-08T10:57:48.000Z LL-994-user-export-department-and-position-missing