Skip to content

feat(order-forms): add OrderForm GraphQL API#5361

Draft
toommz wants to merge 2 commits into
feat/order-forms/read-order-form-restfrom
feat/order-forms/read-order-form-graphql
Draft

feat(order-forms): add OrderForm GraphQL API#5361
toommz wants to merge 2 commits into
feat/order-forms/read-order-form-restfrom
feat/order-forms/read-order-form-graphql

Conversation

@toommz
Copy link
Copy Markdown
Contributor

@toommz toommz commented Apr 17, 2026

Roadmap Task

👉 Order Forms lifecycle

Context

This is PR 2 of 2 in a stacked split. Stacked on the REST PR so the shared model, query, and factories are available. Requires a frontend dev to QA via the UI.

Stack:

Description

Add OrderForm read-only GraphQL API:

  • Types::OrderForms::Object with customer, quote, and lifecycle fields
  • Types::OrderForms::StatusEnum, Types::OrderForms::VoidReasonEnum
  • Resolvers::OrderFormResolver (single) and Resolvers::OrderFormsResolver (collection with filters)
  • Register order_form and order_forms on QueryType
  • Regenerated schema.graphql and schema.json
  • Resolver specs covering single lookup and each filter

@toommz toommz force-pushed the feat/order-forms/read-order-form-graphql branch from 345614f to 56f0bee Compare April 20, 2026 09:48
@toommz toommz self-assigned this Apr 20, 2026
@toommz toommz force-pushed the feat/order-forms/read-order-form-rest branch 5 times, most recently from c56f4a3 to e2f97e2 Compare May 21, 2026 10:19
@toommz toommz force-pushed the feat/order-forms/read-order-form-graphql branch 2 times, most recently from 1c9f83a to 56a3e24 Compare May 21, 2026 13:58
fix(order-forms): register order_forms:view permission

test(order-forms): align list resolver spec with QuoteVersion model
@toommz toommz force-pushed the feat/order-forms/read-order-form-graphql branch from 56a3e24 to 7b3231d Compare May 21, 2026 15:21
Copy link
Copy Markdown
Collaborator

@vincent-pochet vincent-pochet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

field :created_at, GraphQL::Types::ISO8601DateTime, null: false
field :updated_at, GraphQL::Types::ISO8601DateTime, null: false

dataload_association :quote
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we load the customer as well?

expires_at_from:,
expires_at_to:
},
search_term:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like the path testing the search_term is missing

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.

2 participants