Skip to content

Fix Bybit order status and subscription guidance - #24

Merged
tiagosiebler merged 7 commits into
sieblyio:mainfrom
MiniTiago:docs-bybit-corrections
Nov 9, 2026
Merged

tiagosiebler merged 7 commits into
sieblyio:mainfrom
MiniTiago:docs-bybit-corrections

Conversation

@MiniTiago

@MiniTiago MiniTiago commented Nov 9, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Read recent order status with one realtime request by the returned order ID, including recent closed orders. No automatic history fallback.
  • Keep historical lookup and inconclusive-result guidance separate from the example workflow.
  • Explain subscription confirmation through response and rejection through exception; remove the unreachable rejection branch.
  • Distinguish REST recv_window from WebSocket recvWindow.
  • Preserve original errors and use braced conditionals in the public stream example.

Validation

  • Full three-SDK static build and rendered-guide checks; Bybit rebuilt after the single-request simplification.
  • All eight Bybit examples typechecked against published 4.8.4 and 4.8.5 declarations.
  • Existing content, starter and release checks.
  • Exact six-file remote diff verified.

No new tests, dependency changes or exchange calls.

@MiniTiago
MiniTiago marked this pull request as ready for review November 9, 2026 15:49

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

gg mini me

@tiagosiebler
tiagosiebler merged commit 7d147bc into sieblyio:main Nov 9, 2026
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