Skip to content

feat(offers): upsell endpoint for all fare brands#1142

Merged
milesarmstrong merged 1 commit intoduffelhq:mainfrom
BozhidarTsvetkov:addOffersUpsell
Apr 8, 2026
Merged

feat(offers): upsell endpoint for all fare brands#1142
milesarmstrong merged 1 commit intoduffelhq:mainfrom
BozhidarTsvetkov:addOffersUpsell

Conversation

@BozhidarTsvetkov
Copy link
Copy Markdown
Contributor

@BozhidarTsvetkov BozhidarTsvetkov commented Apr 6, 2026

Adds the upsell endpoint for fetching all fare brands for a given offer. Applicable only to some airlines.


Note

Low Risk
Low risk: adds a new Offers.upsellFares wrapper around a new API endpoint plus a focused unit test, without changing existing offer flows.

Overview
Adds Offers.upsellFares(offerId) to call POST /air/offers/{offerId}/upsell, returning a list of offers representing different fare brands for the same itinerary.

Extends Offers.spec.ts with a new nock-backed test that validates the client returns multiple offers (e.g., basic + first class) from the upsell endpoint.

Reviewed by Cursor Bugbot for commit 6d800c9. Bugbot is set up for automated code reviews on this repo. Configure here.

@BozhidarTsvetkov BozhidarTsvetkov requested a review from a team as a code owner April 6, 2026 12:32
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit a7f3a95. Configure here.

Copy link
Copy Markdown
Contributor

@igorp1 igorp1 left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution @BozhidarTsvetkov

@milesarmstrong milesarmstrong merged commit e0ad604 into duffelhq:main Apr 8, 2026
5 checks passed
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