search

bitfinexcom / bfx-api-node-rest

MIT License
12 stars 31 forks source link

replace legacy api for symbols/symbolsDetails #23

Open gasolin opened 5 years ago

gasolin commented 5 years ago

To help reduce platform load, please use https://api.bitfinex.com/v2/conf/pub:list:currency,pub:map:currency:label,pub:map:currency:unit,pub:list:pair:exchange

instead of v1 legacy request in https://github.com/bitfinexcom/bfx-api-node-rest/blob/master/lib/rest2.js#L858

f3rno commented 5 years ago

@gasolin this would change the returned data for symbol details; currently this is the structure:

  {
    "pair": "eususd",
    "price_precision": 5,
    "initial_margin": "30.0",
    "minimum_margin": "15.0",
    "maximum_order_size": "30000.0",
    "minimum_order_size": "2.0",
    "expiration": "NA",
    "margin": false
  }

Can you confirm this is OK?

gasolin commented 5 years ago

I think its not right to return different things that broken the existing integration.

If provide equivalent through v2 API is not possible, maybe it worths to create an alternative v2 API (if make sense)

f3rno commented 5 years ago

@gasolin makes sense, I will forward internally

f3rno commented 5 years ago

Pending backend update.