Skip to content

fix: add fare brand name to order change offer slices#1125

Merged
igorp1 merged 2 commits intomainfrom
cursor/TSP-1276-order-change-offer-farebrandname-f948
Mar 19, 2026
Merged

fix: add fare brand name to order change offer slices#1125
igorp1 merged 2 commits intomainfrom
cursor/TSP-1276-order-change-offer-farebrandname-f948

Conversation

@igorp1
Copy link
Copy Markdown
Contributor

@igorp1 igorp1 commented Mar 18, 2026

Summary

  • add fare_brand_name to OrderChangeOfferSlice so order change offers expose the fare brand in the JS client
  • update order change offer mocks and spec coverage to include the new field
  • propagate the fixture update to related order change request and order change mocks used by type-checked tests

Testing

  • ./node_modules/.bin/jest --runInBand src/booking/OrderChangeOffers/OrderChangeOffers.spec.ts
  • ./node_modules/.bin/tsc --noEmit
  • yarn test (passes as part of the attempted pre-commit hook)

Notes

  • yarn build:test still fails in this environment during the existing Rollup step because rollup.config.mjs uses assert { type: 'json' }, which Node 22 rejects. The TypeScript portion completes successfully before that unrelated failure.

Linear Issue: TSP-1276

Open in Web Open in Cursor 

@igorp1 igorp1 marked this pull request as ready for review March 19, 2026 12:33
@igorp1 igorp1 requested a review from a team as a code owner March 19, 2026 12:33
@igorp1 igorp1 merged commit 1c3b0c8 into main Mar 19, 2026
4 checks passed
@igorp1 igorp1 deleted the cursor/TSP-1276-order-change-offer-farebrandname-f948 branch March 19, 2026 14:03
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.

3 participants