Skip to content

Update the API contract for Trade History#17

Merged
jwalcorn merged 1 commit into
masterfrom
api-test
Nov 5, 2025
Merged

Update the API contract for Trade History#17
jwalcorn merged 1 commit into
masterfrom
api-test

Conversation

@rtclauss

@rtclauss rtclauss commented Sep 9, 2025

Copy link
Copy Markdown
Contributor

This is the corresponding broker PR for IBMStockTrader/trade-history#72

This switches broker to use the updated API contract for trade-history, including the new client classes.

@sonarqubecloud

sonarqubecloud Bot commented Sep 9, 2025

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@rtclauss rtclauss requested a review from anandDev77 November 3, 2025 15:51

@jwalcorn jwalcorn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good. Thanks!

@jwalcorn jwalcorn merged commit a32c13b into master Nov 5, 2025
1 of 2 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.

2 participants