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

adambit追加 #7

Merged
merged 5 commits into from
Dec 21, 2021
Merged

adambit追加 #7

merged 5 commits into from
Dec 21, 2021

Conversation

noy4
Copy link

@noy4 noy4 commented Oct 21, 2021

先方からもらった差分をもとに桑村が代理でPRを出します。

参考スレッド
https://ango-ya.slack.com/archives/C0222A7LMEK/p1634784695000300?thread_ts=1634739179.464900&cid=C0222A7LMEK

@noy4 noy4 requested review from ayapi and nohtaray October 21, 2021 14:11
@noy4 noy4 self-assigned this Oct 21, 2021
@rei0101b
Copy link

ありがとうございますっ

js/adambit.js Outdated
'markets': {
'BTC/ETH': { 'id': 'btc_eth', 'symbol': 'BTC/ETH', 'base': 'BTC', 'quote': 'ETH', 'baseId': 'btc', 'quoteId': 'eth' },
'BTC/USDT': { 'id': 'btc_usdt', 'symbol': 'BTC/USDT', 'base': 'BTC', 'quote': 'USDT', 'baseId': 'btc', 'quoteId': 'usdt' },
'BTC/XRP': { 'id': 'btc_xrp', 'symbol': 'BTC/XRP', 'base': 'BTC', 'quote': 'XRP', 'baseId': 'btc', 'quoteId': 'xrp' },
Copy link
Contributor

Choose a reason for hiding this comment

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

全体的に「ETH 建で BTC を売買する」「XRP 建で BTC を売買する」のような設定になっているようです。
大多数の取引所は base と quote が逆の「BTC 建で ETH を売買する」ような形が多いですが、想定通りの設定になっていますか?

js/adambit.js Outdated
await this.loadMarkets ();
const market = this.market (symbol);
if (price === undefined) {
throw new InvalidOrder (this.id + ' createOrder() requires a price argument for both market and limit orders');
Copy link
Contributor

Choose a reason for hiding this comment

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

成行注文をするために、price を指定しなくてもリクエストできるようにしていただきたいです。

@rei0101b
Copy link

ありがとうございます!
共有いたします!

- fetchOpenOrders では取引履歴ではなく「未約定かつ未キャンセルの自分の注文」が取れること
- IP制限解除
他
@noy4 noy4 merged commit 7541bd6 into feature/btse Dec 21, 2021
@noy4 noy4 deleted the feature/adambit branch December 21, 2021 09:00
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.

4 participants