-
Notifications
You must be signed in to change notification settings - Fork 7
Description
Фидбек №1
В сборке не хватает пакета nuxt в yarn. В остальном все работает через docker run.
Код хорошо разделен на слои. Нейминг подходящий. Обработаны не все ошибки и не хватает тестирования.
Визуализация сделана при помощи приложения на JS, которое общается с сервером через GRPC
Ошибки обрабатываются, но не всегда. Логирования очень мало, из-за этого в ошибочных ситуациях не понятно что происходит и почему не отработал тот или иной вызов. Рекомендуем доработать данный функционал
Документация исчерпывающая
Гибкое создание и настройка стратегии. Добавлено несколько индикаторов.
Фидбек №2
Нерабочая сборка, нужно доставить пакет nuxt, но из-за того что образ залит на dockerhub, можно работать и так. Не понятно зачем нужен энвой. Усложнено, что все в одном контейнере работает.
Нужно добавить тестирование как минимум stackfunсs, а также обработку ошибок. Часто возникала ситуация что при совершении действий ничего не происходило, при этом ни логов ни ошибок на фронте не было.
Визуализация прекрасная. Хочется увидеть сводную страничку по стратегиям, как работают и какие действия совершаются.
Ошибки обрабатываются, но не всегда. Логирования очень мало, из-за этого в ошибочных ситуациях не понятно что вообще происходит и почему не отработал тот или иной вызов
Документация исчерпывающая.