Skip to content

(feat) add validation to candles intiialization#119

Merged
cardosofede merged 1 commit intomainfrom
feat/improve_candles_polling_process
Feb 6, 2026
Merged

(feat) add validation to candles intiialization#119
cardosofede merged 1 commit intomainfrom
feat/improve_candles_polling_process

Conversation

@cardosofede
Copy link
Copy Markdown
Contributor

No description provided.

@rapcmia
Copy link
Copy Markdown
Contributor

rapcmia commented Feb 4, 2026

commit 47d25c0

  • Use 20260201.dev1 and build docker image successfully ✅
  • Added xprl using add-credential route
  • Setup with condor and build generic.grid_strike ✅
    image
    image
    • compare with main branch
      • same behavior after creating the generic.grid_strike. No found errors on logs related to candles
      • deployed bot as well, same behavior but different connector performance since the PR uses 7975

Test with market-data routes, specifically candles related

  • /market-data/candles xrpl XRP-RLUSD ✅
    curl -u admin:admin -X POST -H "Content-Type: application/json" -d '{"connector_name":"xrpl","trading_pair":"XRP-RLUSD","interval":"1m","max_records":50}' http://localhost:8000/market-data/candles | jq
    {
      "detail": "Unsupported connector 'xrpl'. Available connectors: ['binance_perpetual', 'binance', 'bitget', 'bitget_perpetual', 'gate_io', 'gate_io_perpetual', 'kucoin', 'kucoin_perpetual', 'ascend_ex', 'okx_perpetual', 'okx', 'kraken', 'mexc', 'mexc_perpetual', 'bybit', 'bybit_perpetual', 'hyperliquid', 'hyperliquid_perpetual', 'dexalot', 'bitmart_perpetual', 'btc_markets']"
    }
    logs: 11:45:36.680 POST /market-data/candles
    
  • /market-data/historical-candles xrpl XRP-RLUSD 1h window ✅
    curl -u admin:admin -X POST -H "Content-Type: application/json" -d '{"connector_name":"xrpl","trading_pair":"XRP-RLUSD","interval":"1m","start_time":$(date -d '"'"'1 hour ago'"'"' +%s),"end_time":$(date +%s)}' http://localhost:8000/market-data/historical-candles | jq
    {
      "detail": "Unsupported connector 'xrpl'. Available connectors: ['binance_perpetual', 'binance', 'bitget', 'bitget_perpetual', 'gate_io', 'gate_io_perpetual', 'kucoin', 'kucoin_perpetual', 'ascend_ex', 'okx_perpetual', 'okx', 'kraken', 'mexc', 'mexc_perpetual', 'bybit', 'bybit_perpetual', 'hyperliquid', 'hyperliquid_perpetual', 'dexalot', 'bitmart_perpetual', 'btc_markets']"
    }
    
  • /market-data/candles invalid pair XRPRLUSD ✅
    curl -u admin:admin -X POST -H "Content-Type: application/json" -d '{"connector_name":"binance","trading_pair":"XRPRLUSD","interval":"1m","max_records":10}' http://localhost:8000/market-data/candles | jq
    {
      "detail": "Invalid trading pair format 'XRPRLUSD'. Expected format: BASE-QUOTE (e.g., BTC-USDT)"
    

@cardosofede cardosofede merged commit f8801a1 into main Feb 6, 2026
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