Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

woofipro: new dex #22196

Merged
merged 106 commits into from May 10, 2024
Merged

woofipro: new dex #22196

merged 106 commits into from May 10, 2024

Conversation

sc0Vu
Copy link
Contributor

@sc0Vu sc0Vu commented Apr 18, 2024

In this PR, I integrated new exchange woofipro https://dex.woo.org.

TODO (woofipro rest apis):

  • public

    • fetchStatus
    • fetchFundingRateHistory
    • fetchFundingRates
    • fetchFundingRate
    • fetchTime
    • fetchTrades
    • fetchCurrencies
    • fetchMarkets
  • private

    • fetchOHLCV
    • fetchOrderBook
    • createOrder
    • cancelOrder
    • fetchOrders
    • fetchOpenOrders
    • fetchClosedOrders
    • fetchOrder
    • fetchOrderTrades
    • fetchMyTrades
    • cancelAllOrders
    • cancelOrders
    • editOrder
    • fetchTradingFees
    • fetchBalance
    • fetchLeger
    • setLeverage
    • fetchLeverage
    • fetchPosition
    • fetchPositions
    • withdraw
    • createOrders

Test:

  • static tests
  • test shell script

TODO (woofipro ws):

  • public

    • watchOHLCV
    • watchOrderBook
    • watchTicker
    • watchTickers
  • private

    • watchTrades
    • watchOrders
    • watchPositions
    • watchBalance

@sc0Vu sc0Vu requested a review from frosty00 as a code owner May 7, 2024 12:22
this.safeFloat (data, 'close'),
this.safeFloat (data, 'volume'),
];
this.ohlcvs[symbol] = this.safeValue (this.ohlcvs, symbol, {});
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should avoid this type of initialization because the variable loses the type

Copy link
Contributor Author

@sc0Vu sc0Vu May 9, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we did this in other exchanges... should we change to safeDict?

ts/src/pro/woofipro.ts Outdated Show resolved Hide resolved
ts/src/pro/woofipro.ts Outdated Show resolved Hide resolved
@carlosmiei
Copy link
Collaborator

WOOFiPRO_Primary_s-resized

@carlosmiei carlosmiei merged commit 5d37694 into ccxt:master May 10, 2024
1 of 2 checks passed
kroitor pushed a commit that referenced this pull request May 10, 2024
* woofipro: initialize new dex

* woofipro: add fetchMarkets

* woofipro: add fetchCurrencies

* woofipro: add fetchTrades

* woofipro: update

* woofipro: add fetchTime

* woofipro: add fetchFundingRate

* woofipro: add fetchFundingRates

* woofipro: add fetchFundingRateHistory

* woofipro: update

* woofipro: update signature

* woofipro: add fetchOrderBook

* woofipro: add fetchOHLCV

* woofipro: add createOrder

* woofipro: update

* woofipro: add cancelOrder

* woofipro: update

* woofipro: add fetchOrders

* woofipro: add fetchOpenOrders

* woofipro: add fetchClosedOrders

* woofipro: add fetchOrder

* woofipro: update

* woofipro: add fetchOrderTrades

* woofipro: add fetchMyTrades

* woofipro: add cancelAllOrders

* woofipro: add cancelOrders

* woofipro: add editOrder

* woofipro: add fetchTradingFees

* woofipro: update

* woofipro: add fetchBalance

* woofipro: add fetchLeger functions

* woofipro: update

* woofipro: add setLeverage

* woofipro: add fetchLeverage

* woofipro: add fetchPosition

* woofipro: add fetchPositions

* woofipro: updatw

* woofipro: update

* woofipro: update

* woofipro: update

* woofipro: update

* woofipro: update crypto

* woofipro: update

* woofipro: update

* woofipro: update

* woofipro: update

* cs: update exchange

* woofipro: update doc

* woofipro: add withdraw

* woofipro: update

* woofipro: update

* woofipro: update skip tests

* woofipro: update static tests

* woofipro: update static tests

* woofipro: update

* woofipro: update

* woofipro: update

* woofipro: update

* woofipro: add ws

* woofipro: add watchTicker and watchTickers

* woofipro: add watchOHLCV

* woofipro: add watchTrades

* woofipro: add watchOrders

* woofipro: update

* woofipro: add watchPositions

* woofipro: update

* woofipro: add watchBalance

* fix ts syntax

* fix transpilation

* add brokerId

* update docs

* fix tests

* improve editOrder

* more static tests

* add symbol to editOrder

* more tests

* woofipro: update

* woofipro: replace uid with accountId

* fix delete request

* woofipro: add createOrders (need to remove keysort of params)

* woofipro: update createOrders

* woofipro: update createOrders

* fix ts syntax

* static tests setup

* fix static tests

* fix c# build

* fix python syntax

* add secret to response tests

* add woofipro id test

* fix idtest

* ws fixes

* add watchMyTrades

* fix args

* add more tests

* woofipro: update default id

* woofipro: update

* woofipro: use watchMultiple

* fixing linting

* fix handleUNtilOption

* add logo

* add referral

* woofipro: update

* woofipro: update doc

---------

Co-authored-by: carlosmiei <[email protected]>

[ci skip]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants