Implement ticket links and blocker state #13

Merged
Twyco merged 2 commits from codex/0.1-kanban-foundation/10-ticket-links-and-blocker-state into main 2026-04-01 15:39:06 +00:00
Owner
No description provided.
Eric reviewed 2026-03-27 20:59:40 +00:00
@@ -54,6 +56,17 @@ class ProjectTicketController extends Controller
'worklogEntries.actor',
]),
'ticketFieldDefinitions' => $project->ticketFieldDefinitions()->get(),
Owner

Auslagern in nh Service anstatt query im Controller

Auslagern in nh Service anstatt query im Controller
Twyco marked this conversation as resolved
@@ -127,0 +153,4 @@
private function linkedTickets(Ticket $ticket): array
{
return TicketLink::query()
->with(['sourceTicket', 'targetTicket', 'ticketLinkType'])
Owner

Hier ebenfalls auslagern in Service

Hier ebenfalls auslagern in Service
Twyco marked this conversation as resolved
@@ -0,0 +25,4 @@
Ticket $ticket,
): RedirectResponse {
abort_unless(
$this->projectPermissionService->userHasPermission($request->user(), $project, 'edit'),
Owner

Immer dieses abort unless gibts da nicht nh lösung das einmal irg wo zu schreiben, damit man das nicth immer wiederholen muss?
Können wir nicht nh Middleware schreiben die das auch macht oder zu nervig oder unnötig?

Immer dieses abort unless gibts da nicht nh lösung das einmal irg wo zu schreiben, damit man das nicth immer wiederholen muss? Können wir nicht nh Middleware schreiben die das auch macht oder zu nervig oder unnötig?
Author
Owner

Da ist ja jedes mal was anderes im abort_unless

Da ist ja jedes mal was anderes im abort_unless
Twyco marked this conversation as resolved
@@ -41,1 +42,4 @@
->name('projects.tickets.update');
Route::post('projects/{project}/tickets/{ticket}/links', [ProjectTicketLinkController::class, 'store'])
->name('projects.tickets.links.store');
Route::delete('projects/{project}/tickets/{ticket}/links/{ticketLink}', [ProjectTicketLinkController::class, 'destroy'])
Owner

Um sicher zu gehen mit scopedBindings Arbeiten anstatt nicht zu prüfen?

Um sicher zu gehen mit scopedBindings Arbeiten anstatt nicht zu prüfen?
Author
Owner

was rauchst du bei den reviews?

was rauchst du bei den reviews?
Twyco marked this conversation as resolved
Twyco changed target branch from codex/0.1-kanban-foundation/09-ticket-hierarchy-parent-child-relationships to main 2026-03-30 22:07:39 +00:00
Twyco added 1 commit 2026-03-30 22:07:39 +00:00
Twyco added 1 commit 2026-03-30 22:09:37 +00:00
Merge branch 'main' into codex/0.1-kanban-foundation/10-ticket-links-and-blocker-state
linter / quality (pull_request) Successful in 1m32s
tests / ci (8.4) (pull_request) Successful in 1m18s
18785e954e
Twyco merged commit 4a7f3b7cf6 into main 2026-04-01 15:39:05 +00:00
Twyco deleted branch codex/0.1-kanban-foundation/10-ticket-links-and-blocker-state 2026-04-01 15:39:06 +00:00
Sign in to join this conversation.
No Reviewers
No labels
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: PetschStudios/KanbanGit#13