Skip to content

feat(v1.2.3): Advanced Trade static REST sandbox + tutorial accuracy - #142

Merged
tiagosiebler merged 3 commits into
sieblyio:masterfrom
JJ-Cro:fix/coinbase-tutorial-accuracy
Jul 28, 2026
Merged

feat(v1.2.3): Advanced Trade static REST sandbox + tutorial accuracy#142
tiagosiebler merged 3 commits into
sieblyio:masterfrom
JJ-Cro:fix/coinbase-tutorial-accuracy

Conversation

@JJ-Cro

@JJ-Cro JJ-Cro commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Wire Advanced Trade static REST sandbox: useSandbox: true on CBAdvancedTradeClient now routes to https://api-sandbox.coinbase.com (was NoSandboxAvailable).
  • Bump version to 1.2.3.
  • Update tutorial: sandbox hosts, FAQ, and useSandbox example. Keep no Advanced Trade WebSocket sandbox.
  • Keep prior tutorial accuracy fixes (sync subscribe() / exception handling, heartbeats signing note).
  • No X-Sandbox variance-header helper in this PR.

Test plan

  • new CBAdvancedTradeClient({ useSandbox: true }) hits https://api-sandbox.coinbase.com
  • Live AT still defaults to https://api.coinbase.com
  • Exchange / INTX sandbox hosts unchanged
  • Advanced Trade WS useSandbox still resolves to unavailable
  • Tutorial sandbox section matches SDK behavior

JJ-Cro and others added 2 commits July 27, 2026 18:49
Document Advanced Trade useSandbox stub vs explicit baseUrl, note client-wide useSandbox, remove misleading try/catch around sync subscribe, and clarify heartbeats signing on the user-data feed.

Co-authored-by: Cursor <cursoragent@cursor.com>
Map CBAdvancedTradeClient useSandbox to https://api-sandbox.coinbase.com, update tutorial and option docs. No X-Sandbox header helper yet; Advanced Trade WebSocket sandbox remains unavailable.

Co-authored-by: Cursor <cursoragent@cursor.com>
@JJ-Cro JJ-Cro changed the title fix: clarify Coinbase quickstart sandbox and subscribe error handling feat(v1.2.3): Advanced Trade static REST sandbox + tutorial accuracy Jul 28, 2026
Updated table formatting for Advanced Trade channels and API environments for improved readability. Ensured consistent alignment and spacing across sections.

@tiagosiebler tiagosiebler 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.

LGTM

@tiagosiebler
tiagosiebler merged commit f3b33c8 into sieblyio:master Jul 28, 2026
1 check 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