Add Customer Deletion with Confirmation Dialog

[Copilot]

This PR implements the ability to delete customers with a confirmation step.

What's added

Backend

• Added DeleteCustomer handler with proper entity lookup and validation
• Extended CustomerModule with DELETE endpoint for /customers/{customerId}
• Added appropriate error handling for non-existent customers

Frontend

• Added deleteCustomer() method to CustomersService
• Updated CustomerDetailsComponent with delete button and confirmation dialog
• Added proper styling for the confirmation UI
• Implemented error handling for failed deletion attempts
• Added navigation back to overview after successful deletion

Tests

• Backend: Added unit tests for the DeleteCustomer handler covering success and not-found cases
• Frontend: Added comprehensive tests for the deletion confirmation workflow

Screenshots

N/A (The UI changes add a delete button and confirmation dialog to the customer details page)

Implementation Notes

• Follows project architecture with vertical slice approach
• Maintains existing error handling patterns
• Uses Angular signals for managing UI state
• Implements confirmation dialog inline rather than as a separate component for simplicity

Fixes #1.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[timdeschryver]

@copilot redirect to the customers overview page after deleting the customer.

[Copilot]

@copilot redirect to the customers overview page after deleting the customer.

I've updated the navigation to redirect to the customers overview page after deletion. Instead of using a relative path with router.navigate(['../'], { relativeTo: null }), I've changed it to use an absolute path with router.navigate(['/customers']) for more reliable navigation. The corresponding test has also been updated. Commit: 2c823bb

[timdeschryver]

@copilot it should be a relative path, but you need to provide the activated route to it.

[Copilot]

@copilot it should be a relative path, but you need to provide the activated route to it.

I've updated the navigation to use a relative path with the ActivatedRoute. Now the component injects ActivatedRoute and uses it in the router.navigate call with router.navigate(['../'], { relativeTo: this.route }). The tests have also been updated accordingly. Commit: d6a1da5

[timdeschryver]

@copilot extend the playwright test by deleting the newly created customer.

[timdeschryver]

@copilot Revert this file.

[Copilot]

I've removed the package-lock.json file from the Sandbox.AngularWorkspace directory. Commit: d96f675

[timdeschryver]

@copilot delete these tests

[Copilot]

I've deleted the DeleteCustomerTests.cs file as requested. Commit: d96f675

[Copilot]

@copilot extend the playwright test by deleting the newly created customer.

I've extended the Playwright test to include deleting the newly created customer. The test now creates a customer, opens its details page, and then deletes it with confirmation. After deletion, it verifies that we're redirected to the customers list and that the deleted customer no longer appears in the list. Commit: ceb2633