search

Tinkoff / invest-openapi-java-sdk

Apache License 2.0
167 stars 46 forks source link

Улучшение интервалов по свечам #143

Closed Freeongoo closed 2 years ago

Freeongoo commented 3 years ago
  1. Устранить два одинаковых Enum: ru.tinkoff.invest.openapi.model.res.CandleResolution и ru.tinkoff.invest.openapi.model.stream.CandleInterval, не вижу ни одной причины зачем использовать два Enum.
  2. Использовать более расширенный Enum ru.tinkoff.invest.openapi.model.stream.CandleInterval в качестве основного, поскольку в ru.tinkoff.invest.openapi.model.res.CandleResolution нет интервала 4hour а он очень нужен для технического анализа.
zlumyo commented 2 years ago

Enum из пакета model.rest сгенерирован. Придётся отказаться от генерации моделей из сваггера целиком. Расширенный Enum использовать не совсем корректно, поскольку при использовании 4hour будет возникать ошибка. Это проблема скорее на уровне API, а не SDK. Думаю, лучше обратиться с этим вопросом в соответствующий репозиторий. Хотя сейчас готовится к выпуску API версии 2.0, где всё поменяется, поэтому вряд ли в текущей версии эту проблему устранят.