omsdotnet / bookmakerboard Goto Github PK

В рамках подготовки к рефакторингу

Для получения гарантий работоспособности приложения и повышения качества предлагаю покрыть интеграционными тестами класс Program.

За данными обращаться в контроллеры Bidders и Rides

после аутентификации становятся доступны все формы CRUD, без регистрации - только основное табло

Does this project use any Continuous Integration Tool? If so, is it online? (TravisCI, CircleCI), or offline tool (Jenkins, Hudson).

В текущей реализации, API не удовлетворяет требованиям REST, не выполняются необходимые ограничения накладываемые стилем. Необходимо исправить маршруты, модели, добавить HATEOAS.

для старта достаточно только одного пользователя

Контролеры: BiddersController, RidesController, TeamsController не работают с View и по сути являются контроллерами API.
Необходимо привести в соответствие указанные контроллеры, заменив базовый класс на ControllerBase и добавив атрибут ApiController.

Механизм с зашифрованным паролем не безопасен. Кроме того, это препятствует созданию многопользовательского режима.
Необходимо удалить текущую реализацию аутентификации в пользу стандартных обработчиков. Кроме того необходимо добавить внешних поставщиков авторизации. Например GitHub OAuth 2.0.

Текущая реализация хранилища требует разрешения зависимостей в классе Startup во время конфигурации служб.
Разрешение зависимостей должно выполняться один раз, в момент запуска приложения.
Для исправления необходимо реализовать IHostedService который будет осуществлять загрузку и выгрузку данных источника.

В настоящий момент проект создан на платформе .NET Core 2.1. Необходимо обновить версию платформы и соответствующие скрипты для сборки решения.

Для запуска модульных и интеграционных тестов для frontend приложения необходимо добавить в workflow sonarcloud-analysis шаг запускающий команду npm test...

регистрирует букмейкеров админ сайта, те могут подавать заявки на регистрацию

Пользователь может предложить свою ставку букмейкеру, а он может либо принять ее либо отклонить

В текущей реализации CI используется Cake и AppVeyor.
Cake скрипт сложно разрабатывать и поддерживать, плюс часть скрипта связанная с sonarqube устарела (версия сервера намного выше и текущая версия инструментария не может взаимодействовать с сервером).
Необходимо отказаться от поставщика AppVeyor, реализовать CI в GitHub Actions и избавиться от Cake скриптов в пользу dotnet CLI и PowerShell.

Для последующей разработки предлагаю внедрить статическую типизацию используя TypeScript.