T. Todua
T. Todua
as automatic inclusion (of test/base files) at this moment is not approved, closing this for now.
i've included the unmerged commits from https://github.com/carlosmiei/ccxt/tree/waves-fix
Can anyone give some step-by-step instruction how to build android/ios app using this? -- edit: found [this article](https://medium.com/@acejon96/building-an-android-app-with-vue-js-fc0aefb454d9), might help, or [this](https://github.com/mahbuburriad/Convert_Any_website_into_android_app) to serve it through remote .html file;
@bgparkloop after the linked PR is merged, the issue should be gone. however, in future, you will be able to control hedged position mode by passing `params = {'hedged': True/False}`...
@carlosmiei is right, hiding it is not a good approach, we definitely have to offer an option so user could control it. i'll be adding this shortly.
I've linked PR. after it's merged and new package published, you can use `ex.options['usePrivateEndpointsIfApikeysPresent'] = true` (the keyname might be changed before merging, keep an eye on it), so thereafter...
the `average` field should be present, also `cost` field has been added. this issue is going to be closed now, if any further issues, feel free to re-open
i'll add static tests
i decided to refrain from adding static tests, as they involve more changes (changes in test.ts too) to handle apiKey & secret fields, but i have pending 'tests reorg PR'...
@yyqiekenao as commented by @samgermain it would be ideal if you try to enable verbose mode and provide us what raw-data is coming in which causes those nulled valued.