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

Обновление версии протокола #24

Closed
Jokky opened this issue Dec 18, 2024 · 5 comments
Closed

Обновление версии протокола #24

Jokky opened this issue Dec 18, 2024 · 5 comments

Comments

@Jokky
Copy link

Jokky commented Dec 18, 2024

В данный момент тиньк дает версию 1.24, можно ли как-то самостоятельно обновить и опубликовать изменения?

Новый репозиторий документации: https://github.com/RussianInvestments/investAPI

@vitalets
Copy link
Owner

Да, можно попробовать в форке обновить proto-файлы, вот тут обсуждали процесс: #15
Скорее всего будут какие-то ошибки, если что приносите в этот тикет, посмотрим.

@Jokky
Copy link
Author

Jokky commented Dec 18, 2024

Обновил прото в отдельной ветке и возникли следующие ошибки. С первыми двумя еще понятно как разобраться, но проблема есть еще с candles-loader/req.ts. В таком случае будет правильно передавать пустую строчку для кэширования?
12490

@Jokky
Copy link
Author

Jokky commented Dec 18, 2024

@vitalets #25

@vitalets
Copy link
Owner

С первыми двумя еще понятно как разобраться, но проблема есть еще с candles-loader/req.ts. В таком случае будет правильно передавать пустую строчку для кэширования?

instrumentId либо figi всегда должен быть. Поэтому предлагаю так:

protected getInstrumentKey() {
   const key = this.params.instrumentId || this.params.figi;
   if (!key) throw new Error(`instrumentId or figi is required.`);

   return key;
}

@vitalets
Copy link
Owner

Зарелизил 7.0.0.
Спасибо!

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

2 participants