Details des Tickets

Beschreibung,Kommentare und Anhänge

merge request id 180
TitelFeature/ll 310 add permissions for reports
BeschreibungDie neuen Rechte werden beim Deploy-Vorgang mit eingelesen und der Rolle Mandant zugeordnet
Statusmerged
Created at 2020-05-05T10:37:47.428Z
Updated at 2020-05-13T11:33:23.060Z
Source branchfeature/LL-310-add-permissions-for-reports
Target branchdevelop
AuthorGregor Gabriel
AssigneeMarcel Pociot

Discussion notes

assigned to @schleins Sascha Immig 2020-05-08T08:12:53.774Z
assigned to @pociotm Sascha Immig 2020-05-12T06:42:57.160Z
@gabrielg Wieso hast du den Seeder mit ins deployment hinzugefügt? Aktuell versucht der Seeder jedes mal die Rollen anzulegen - also sollte man den eigentlich nur beim aller ersten mal aufrufen. Marcel Pociot 2020-05-12T08:11:23.761Z
die Idee war, dass der Seeder als Updater fungiert... der Seeder wurde ja von `create` auf `createorupdate` angepasst... Sascha Immig 2020-05-12T08:13:47.800Z
Ah okay.. das habe ich im diff übersehen, weil die Änderungen so groß waren! :thumbsup: Marcel Pociot 2020-05-12T08:19:38.852Z
Idee ist, dass man damit jederzeit weitere Rechte und Rollen bereitstellen kann, zentral über diesen Seeeder und nicht jedes mal eine Migration schreiben muss. Gregor Gabriel 2020-05-12T08:59:10.974Z
Wenn das i.O ist mit dem Seeder-Umbau... @gabrielg bitte noch den Merge-Konflikt entfernen (sind wohl neue Rechte in den Seeder gekommen) @pociotm wenn die Konflite weg sind bitte mergen. Danke :) Sascha Immig 2020-05-12T12:19:16.254Z
Ja für mich ist der MR dann auch in Ordnung und kann gemerged werden Marcel Pociot 2020-05-12T14:09:30.724Z
added 52 commits
  • 100fc101...1500852b - 51 commits from branch develop
  • 1ac73ffd - Merge branch 'develop' of https://gitlab.learningsystem.de/lms/seventeen into…
Compare with previous version
Gregor Gabriel 2020-05-12T15:13:37.260Z
Hab's gemerged Gregor Gabriel 2020-05-12T15:20:36.088Z
merged Sascha Immig 2020-05-13T11:33:23.103Z
mentioned in commit e7baf1240d00b7abfda97bce344b2673ef5c9491 Sascha Immig 2020-05-13T11:33:23.291Z

Merge Commits

Title Id Author Created at Message
make sure premissions anf roles are updated in deployment process 100fc1013cb5047ba41a81c73386e335abe59acd TUV\gabrielg 2020-05-05T10:35:26.000Z make sure premissions anf roles are updated in deployment process
Roles and permissions seeder can now be executed multiple times 17fe1ecce9603ae3bf97e47153e63ed3e2192bdd TUV\gabrielg 2020-05-05T08:02:44.000Z Roles and permissions seeder can now be executed multiple times
Merge branch 'develop' of https://gitlab.learningsystem.de/lms/seventeen into… 1ac73ffde0de9560aabb6627beb2812c441821bf TUV\gabrielg 2020-05-12T15:13:31.000Z Merge branch 'develop' of https://gitlab.learningsystem.de/lms/seventeen into feature/LL-310-add-permissions-for-reports # Conflicts: # Envoy.blade.php # app/Enums/Permissions.php # database/seeds/RolesAndPermissionsSeeder.php # resources/lang/de.json
+ added missing permissions for new reporting 372ce6ee291f95d02c48bbaa8c6aaca65579faf6 TUV\gabrielg 2020-05-05T10:06:37.000Z + added missing permissions for new reporting + added new reporting permissions to client role + new permissions will be available by calling "php artisan db:seed --class=RolesAndPermissionsSeeder"
added missing classes and extends 48a428e6c7ef3b49313b2f62c4da6f2ece825150 TUV\gabrielg 2020-05-05T10:15:54.000Z added missing classes and extends