Skip to content

Conversation

@vitalets
Copy link
Contributor

@vitalets vitalets commented Jun 6, 2022

Сделал отдельный ПР с Tinkoff API v2 из своего форка.

  • пока перевел на api v2 только transports/tinkoff.v2.ts (там вопросики в коде). transports/tinkoff.ts оставил для референса.
  • добавил тесты

@vitalets
Copy link
Contributor Author

vitalets commented Jun 8, 2022

Все поправил, кроме ретраев.
@BusinessDuck , предлагаю этот пр вмерджить, а ретраи отдельно докатить, чтобы смотреть удобнее было. Что скажешь?

И потом отдельно history-providers/tinkoff сделаю.

@BusinessDuck
Copy link
Member

Все поправил, кроме ретраев. @BusinessDuck , предлагаю этот пр вмерджить, а ретраи отдельно докатить, чтобы смотреть удобнее было. Что скажешь?

И потом отдельно history-providers/tinkoff сделаю.

Не нужно мержить, пока не закончен history, давай его добьем и вмержим. А я пойду тестировать запросы истории, потому что там много крайних кейсов, например, время включительно не включительно. Для Быстроти провайдер истории бьет запрос на 12 часовые интервалы и результат склеивает по дням, нужно проверять что это работает, оставь это мне, с тебя только сетевые запросы

jest.config.ts Outdated

// The paths to modules that run some code to configure or set up the testing environment before each test
// setupFiles: [],
// setupFiles: ['dotenv/config'],
Copy link
Member

Choose a reason for hiding this comment

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

остаточное

@BusinessDuck
Copy link
Member

Все поправил, кроме ретраев. @BusinessDuck , предлагаю этот пр вмерджить, а ретраи отдельно докатить, чтобы смотреть удобнее было. Что скажешь?
И потом отдельно history-providers/tinkoff сделаю.

Не нужно мержить, пока не закончен history, давай его добьем и вмержим. А я пойду тестировать запросы истории, потому что там много крайних кейсов, например, время включительно не включительно. Для Быстроти провайдер истории бьет запрос на 12 часовые интервалы и результат склеивает по дням, нужно проверять что это работает, оставь это мне, с тебя только сетевые запросы

Кстати просмотрел history, возможно его прям не сложно будет сделать, тк запросы и в моей ветке были почти закончены

@vitalets
Copy link
Contributor Author

vitalets commented Jun 8, 2022

Доделал history, прямо в history/tinkoff.ts

@BusinessDuck
Copy link
Member

Мержу и начинаю проверки

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.

2 participants