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

Возможность коннектиться к разным маркетам #59

Closed
2 tasks done
youaresofunny opened this issue Apr 3, 2018 · 9 comments
Closed
2 tasks done
Assignees

Comments

@youaresofunny
Copy link
Contributor

youaresofunny commented Apr 3, 2018

Сейчас service market работает как singleton (export default new Market('1.3.0'))

Нужно:
Возвращать объект где ключ это base asset, а значение - экземпляр в маркете.

Тогда в modules обойдёмся малой кровью:

API.Market['1.3.0'].subscribeToExchangeRate...
API.Market['1.3.0'.unsubscribeFromExchangeRate

И на экране мануальных ордеров будем забирать стакан из выбранной базы путем подписывания на нужный ассет.

TODO:

  • Поправить services/market чтобы на выходе был объект с синглтонами
  • Поправить места которые использовали этот маркет, чтобы они обращались к BTS
@roma219
Copy link
Contributor

roma219 commented Apr 3, 2018

Не уверен, что стоит таким громоздким способом это делать. Для начала надо понять, а какие сценарии использования других маркетов предполагаются? Необходима ли возможность использования двух маркетов одновременно?

@youaresofunny
Copy link
Contributor Author

@roma219 необходима, когда юзер будет торговать на manual табе, в фоне будет висеть BTS Market.

@roma219
Copy link
Contributor

roma219 commented Apr 3, 2018

@youaresofunny что?

@youaresofunny
Copy link
Contributor Author

"Необходима ли возможность использования двух маркетов одновременно?" - да

@roma219
Copy link
Contributor

roma219 commented Apr 3, 2018

@youaresofunny Ок, это гораздо большая задача, чем просто в API.Market сделать объект с маркетами... Есть подводные камни и в количестве. Думаю целесообразно сначала сделать мануал таб чисто для bts маркета (держа в уме будущее возможность его замены), а далее уже внедрять возможность изменения - тогда и я ясно будет, в какой форме это лучше будет делать.

@youaresofunny
Copy link
Contributor Author

Мануал trading и vuex под него будут в другом issue.
Конкретно здесь все ок - малыми силами расширили service для работы с любым маркетом.

@roma219
Copy link
Contributor

roma219 commented Apr 3, 2018

А зачем сейчас нужно это расширение? Если его использование предполагается только в модуле manual. Да и реализовать сейчас лучше как:

const baseMarket = API.Market.getMarket('1.3.0')
один раз вверху после импорта API

а не городить в каждом модуле API.Market['1.3.0']

@youaresofunny
Copy link
Contributor Author

youaresofunny commented Apr 3, 2018

const baseMarket = API.Market['1.3.0']

@youaresofunny
Copy link
Contributor Author

#60

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

No branches or pull requests

3 participants