Skip to content

feat: support the seller's side of the quote flow - #438

Open
korsvanloon wants to merge 1 commit into
mainfrom
feat/seller-quote-flow
Open

korsvanloon wants to merge 1 commit into
mainfrom
feat/seller-quote-flow

Conversation

@korsvanloon

Copy link
Copy Markdown
Collaborator

The seller's side of quotes was mostly missing: POST /staged-quotes and POST /quotes returned a resource without storing it, a new quote came back Accepted, and none of the update actions a seller uses existed.

  • Staged quotes and quotes are stored on create
  • Staged quote: created from a Submitted request only, checks quoteRequestVersion, prices the offer in a copy of the requested cart, honours quoteRequestStateToAccepted
  • Quote: created Pending from an InProgress staged quote only, checks stagedQuoteVersion, carries validTo, sellerComment, business unit and store, honours stagedQuoteStateToSent
  • Staged quote actions: changeStagedQuoteState, setSellerComment, setValidTo
  • Quote actions: changeQuoteState (refuses DeclinedForRenegotiation), requestQuoteRenegotiation
  • Tests walk the whole flow, including answering a renegotiation with a new quote from the same staged quote

Staged quotes and quotes are stored when created, a quote starts Pending
rather than Accepted, and the staged quote and quote update actions a
seller needs to send, withdraw and renegotiate a quote are implemented.
@changeset-bot

changeset-bot Bot commented Sep 24, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 02400db

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@labdigital/commercetools-mock Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

This branch has not been deployed

No deployments
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