search

vitalets / tinkoff-invest-api

Node.js SDK для работы с Tinkoff Invest API
45 stars 11 forks source link

Use Cache Parameter #3

Closed BusinessDuck closed 2 years ago

BusinessDuck commented 2 years ago

Сделай пожалуйста опциональным сохранение кеша. Мне не оч нравится вообще идея кешировать что-то на стороне библиотеки, кажется что это задача внешнего модуля (Just my mind). Если не убирать то хотябы отключить для дебюта, там свой кеш.

https://github.com/vitalets/tinkoff-invest-api/blob/6171ae90e49098b4f075991625a106602206e400/src/candles-loader/req.ts#L43

vitalets commented 2 years ago

Если не нужен кеш, то можно напрямую из api.marketdata.getCandles() забирать вместо candlesLoader. Или тебе нужна разбивка по чанкам, но без кеша? Просто разбивка по чанкам (дням) же в дебюте есть.

BusinessDuck commented 2 years ago

Если не нужен кеш, то можно напрямую из api.marketdata.getCandles() забирать вместо candlesLoader. Или тебе нужна разбивка по чанкам, но без кеша?

Просто разбивка по чанкам (дням) же в дебюте есть.

если ты напрямую сделал там, то вопросов нет :) можно закрывать тикет

vitalets commented 2 years ago

если ты напрямую сделал там, то вопросов нет :) можно закрывать тикет

Да, там напрямую - просто получение свечей с сервера.