pr improvements
linter / quality (pull_request) Successful in 1m15s
tests / ci (8.4) (pull_request) Successful in 1m27s

This commit is contained in:
2026-04-15 15:46:44 +02:00
parent cec0e397b6
commit 931ae29dea
6 changed files with 18 additions and 43 deletions
+10 -35
View File
@@ -1,37 +1,12 @@
## Missing features
- (mehr über middlewares steuern)
- mehrere actions/quereys in services/actions auslagern
- generell, aber besonders UpdateProjectTicket ist sehr unübersichtlich
- repository_connections check base Url maybe unnessecary?
- Git listBranches replace baseUrl, owner, rperoName with GItReprository
- Git prs replace baseUrl, owner, rperoName with GItReprository
- replace Kanbangit with env App Name
- change Project Backlog Controller → Project Controller
- handle backlog as a normal column -> only difference is an seperate view in frontend /backlog and /board bleibt bestehen
- remove store ticket from backlog -> create ticket create view.
- always show create on board and backlog -> rediurect to ticket create
- ggf worklog überarbeiten? → type als worklog enum → json je nach enum type oder so
- ziel ist einheitlicherere worklog struktur
- falls sinnvoll, sonst verwerfen
- app/Http/Controllers/ProjectTicketController.php parent ids anderes queren. nicht alle child ids berechnen und excluden -> immer parent vom parent bis null
## Umsetzung 2026-04-15
### Umgesetzt
- (mehr über middlewares steuern)
- mehrere actions/quereys in services/actions auslagern
- Git listBranches replace baseUrl, owner, rperoName with GItReprository
- Git prs replace baseUrl, owner, rperoName with GItReprository
- replace Kanbangit with env App Name
- change Project Backlog Controller → Project Controller
- remove store ticket from backlog -> create ticket create view.
- always show create on board and backlog -> rediurect to ticket create
- ggf worklog überarbeiten? → type als worklog enum
- app/Http/Controllers/ProjectTicketController.php parent ids anderes queren. nicht alle child ids berechnen und excluden -> immer parent vom parent bis null
### Nicht umgesetzt
- repository_connections check base Url maybe unnessecary? - `base_url` bleibt nötig, um Self-hosted-Git-Instanzen und passende User-Credentials eindeutig zuzuordnen.
- handle backlog as a normal column - Backlog bleibt bewusst ein Status ohne `board_column_id`; `/backlog` und `/board` sind nur getrennte Views.
- resources benutzen
- Für jedes Model / Route, resources benutzen, kein mapping/feste arrays mehr in den controller.
- kein 'model' => $model (bspw. 'project' => $project) mehr.
- niemals das model in reponse return. für jedes model deshalb eine resource und diese benutzen.
- in der resource niemals mit only oder manuel die props übergeben.
- Ziel: neue attribute von model sollen nicht ausversehen in responses auftauchen & responses kleiner, falls manche attr nicht gebraucht werden
- folgende actions löschen und durch resource benutzen:
- app/Actions/Projects/BuildProjectBacklogPageProps.php
- app/Actions/Projects/BuildProjectBoardPageProps.php
- app/Actions/Tickets/BuildProjectTicketUpdateAttributes.php