Skip to content

Add list-all orders endpoint for the admin orders view#99

Closed
CarsonL15 wants to merge 1 commit into
mainfrom
orders-list-all-endpoint
Closed

Add list-all orders endpoint for the admin orders view#99
CarsonL15 wants to merge 1 commit into
mainfrom
orders-list-all-endpoint

Conversation

@CarsonL15
Copy link
Copy Markdown
Contributor

Summary

Adds an admin-facing "list all orders" capability to the Order Service so the Admin & Maintenance App can show every order (issue #53). Today GET /api/orders requires a customerId; an admin has no way to list orders across customers.

Change

GET /api/orderscustomerId is now optional:

  • omitted → returns all orders (newest first)
  • provided → unchanged, returns that customer's history

Backward compatible: the customer web app's existing ?customerId= calls behave exactly as before.

  • OrdersController.GetOrders branches on whether customerId is supplied
  • IOrderService.GetAllOrdersAsync + implementation (no WHERE CustomerId filter)

Relates to #53 (consumed by the admin Orders view).

@CarsonL15 CarsonL15 requested a review from a team June 2, 2026 20:34
@CarsonL15
Copy link
Copy Markdown
Contributor Author

Consolidating into the Sprint 3 PR instead — both pipelines still trigger by path filter on merge.

@CarsonL15 CarsonL15 closed this Jun 2, 2026
@CarsonL15 CarsonL15 deleted the orders-list-all-endpoint branch June 2, 2026 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant