Skip to content

feat(orders): add Order REST API#5362

Closed
toommz wants to merge 2 commits into
feat/order-forms/read-order-form-graphqlfrom
feat/order-forms/read-order-rest
Closed

feat(orders): add Order REST API#5362
toommz wants to merge 2 commits into
feat/order-forms/read-order-form-graphqlfrom
feat/order-forms/read-order-rest

Conversation

@toommz
Copy link
Copy Markdown
Contributor

@toommz toommz commented Apr 17, 2026

Roadmap Task

👉 Order Forms lifecycle

Context

This is PR 3 of 4 in a stacked split. Stacked on the OrderForm GraphQL PR so all OrderForm code (REST + GraphQL + shared) is available.

Stack:

Description

Add Order read-only REST API plus the shared infrastructure for Order:

  • OrdersController with inlined index and show actions
  • V1::OrderSerializer for REST API serialization
  • OrdersQuery with filters: status, order_type, execution_mode, customer_id, number, order_form_number, quote_number, owner_id, executed_at range
  • Queries::OrdersQueryFiltersContract for filter validation
  • Order factory
  • Route resources :orders, only: %i[show index]
  • ApiKey::RESOURCES updated with order
  • Full specs for serializer, query, controller

@toommz toommz force-pushed the feat/order-forms/read-order-form-graphql branch from 345614f to 56f0bee Compare April 20, 2026 09:48
## Context

Order read-only REST API for listing and showing orders, with
filters (status, order_type, execution_mode, customer_id, number,
order_form_number, quote_number, owner_id, and executed_at range).
Split from the combined read-order branch to unblock REST QA
independently of frontend.

## Description

Add Order model extensions, query object with filters, REST
controller (index and show) with inlined index action, serializer,
and routes. Register order in ApiKey::RESOURCES.
@toommz toommz force-pushed the feat/order-forms/read-order-rest branch from e25f379 to cef8314 Compare April 20, 2026 09:48
@toommz toommz self-assigned this Apr 20, 2026
@toommz toommz closed this May 21, 2026
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