feat(okx): add 15 documented endpoints missing from implicit API#28427
Merged
carlosmiei merged 1 commit intoccxt:masterfrom Apr 19, 2026
Merged
feat(okx): add 15 documented endpoints missing from implicit API#28427carlosmiei merged 1 commit intoccxt:masterfrom
carlosmiei merged 1 commit intoccxt:masterfrom
Conversation
Audit-driven follow-up that fills documented endpoints not present in ts/src/okx.ts. Touches only the api object; no behavior change. Trading bot — Recurring Buy (6, fills out the family): POST tradingBot/recurring/add-investment POST tradingBot/recurring/amend-price-range POST tradingBot/recurring/amend-recurring-amount POST tradingBot/recurring/amend-recurring-time POST tradingBot/recurring/pause POST tradingBot/recurring/restart Trading bot — Grid (1): POST tradingBot/grid/copy-order-algo Public market data (4): GET market/call-auction-details (sibling rename of call-auction-detail) GET public/event-contract/events GET public/event-contract/markets GET public/event-contract/series Account (2): GET account/subtypes POST account/set-riskOffset-amt (sibling of set-riskOffset-type) Stats / support (2): GET rubik/stat/contracts/long-short-position-ratio-contract-top-trader GET support/announcement-types (sibling rename of announcements-types) The two "sibling rename" entries (call-auction-detail and announcements-types) are kept in place and tagged in a comment to avoid breaking existing callers. Closes ccxt#28426 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
lucasjamar
requested review from
carlosmiei,
frosty00 and
kroitor
as code owners
Collaborator
|
@lucasjamar thanks a lot for your contribution! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Audit-driven follow-up to #28425. After grepping all
/api/v5/...paths from the OKX docs HTML and diffing againstts/src/okx.ts, 15 documented endpoints were still missing. Touches onlyts/src/okx.ts(+17 / -2).Trading bot — Recurring Buy (6, fills out the family)
tradingBot/recurring/add-investmentprivatePostTradingBotRecurringAddInvestmenttradingBot/recurring/amend-price-rangeprivatePostTradingBotRecurringAmendPriceRangetradingBot/recurring/amend-recurring-amountprivatePostTradingBotRecurringAmendRecurringAmounttradingBot/recurring/amend-recurring-timeprivatePostTradingBotRecurringAmendRecurringTimetradingBot/recurring/pauseprivatePostTradingBotRecurringPausetradingBot/recurring/restartprivatePostTradingBotRecurringRestartTrading bot — Grid (1)
tradingBot/grid/copy-order-algoprivatePostTradingBotGridCopyOrderAlgoPublic market data (4)
market/call-auction-detailspublicGetMarketCallAuctionDetailspublic/event-contract/eventspublicGetPublicEventContractEventspublic/event-contract/marketspublicGetPublicEventContractMarketspublic/event-contract/seriespublicGetPublicEventContractSeriesmarket/call-auction-details(plural) is the path currently shown in the OKX docs; ccxt has the singularcall-auction-detail. Both are kept, with the old one tagged// deprecated, use call-auction-detailsin a comment, so existing callers are not broken.Account (2)
account/subtypesprivateGetAccountSubtypesaccount/set-riskOffset-amtprivatePostAccountSetRiskOffsetAmtset-riskOffset-amtis the sibling of the already-supportedset-riskOffset-type(set the offset amount vs. the offset type).Stats / support (2)
rubik/stat/contracts/long-short-position-ratio-contract-top-traderpublicGetRubikStatContractsLongShortPositionRatioContractTopTradersupport/announcement-typespublicGetSupportAnnouncementTypessupport/announcement-typesis the path in the current OKX docs; ccxt hassupport/announcements-types(plural — looks like a typo). Both are kept, with the old one tagged in a comment.Out of scope
fiat/buy-sell/*,fiat/{deposit,withdrawal,...}, 13 endpoints) — see [okx] Add 15 documented endpoints missing from implicit API #28426. Held back pending maintainer feedback on whether ccxt accepts fiat additions for OKX.Closes #28426
Test plan
npm run pre-transpileclean (TS compile,emitAPI, ESLint, bundle)ts/src/abstract/okx.tsregenerates with all 15 new method signaturesnew ccxt.okx()and assert every generated method name is a function — 15/15 pass🤖 Generated with Claude Code