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

Список замечаний и доработок шаблона #55

Open
AntonyNET opened this issue Oct 19, 2023 · 1 comment
Open

Comments

@AntonyNET
Copy link

AntonyNET commented Oct 19, 2023

byndyusoft-templates-api

Почему репа называется templates, а не template?
Почему два ридми? Они никак не связаны.

Domain
нет примера сбора метрик, трассировки, активити
нет сервисов IGuidService, IDateTimeService ... (Sod.Shared)
неполная структура папок Entities, Dtos, Mappers......

DataAccess
Byndyusoft.Data.Relational.Abstractions

Migrator
имя миграции не соответсвует конвенциям, включить номер миграции и детали.
В аттрибуте миграции тоже указать description
не хватает расширения для двухфазной миграции.

Api.Client
Удалить Tracer, заменить на активити и листнеры событий.
Добавить Polly
Добавить хелсчек
Не предложена структура папок, как хранить клиенты разных версий.
Из клиентов не понятно, какой клиент для какой версии.
Удалить зависимость от BaseClient, слишком ограничивает

Api
Контроллеры версионированны, но не предложена структура папок, как хранить версии.
переделать хелсчеки, live ready details . Добавить чек на базу, очередь и тд. Таймауты на хелсчеки
Добавить рамширенные ApiConventions, которые будут соответствовать нашим.
Удалить все атрибуты ProducesResponseType и Produces
Куда пропал UseDeveloperExceptionPage?
Не использовать ActionResult
Подключить Scrutor
Не хватает HosttedServic, не понимаю зачем разделено на апи и воркеры, хотя по сути это одно и то же

UnitTests
переименовать в Domain.Tests
не хватает ссылок на расширения Moq для FluentAssertions (Sod.Tests.Shared)
не хватает шаблоном xfact xtheory по конвенциям

IntegrationTests
переименовать в Api.IntegrationTests
не хватает ServiceFixtureBase, ITestOutputLogger, QueueServiceStub (Sod.Tests.Shared.IntegrationTests)
не хватает примера теории и применения концепции
неполная структура папок

byndyusoft-dotnet-test-infrastructure
название репы не соответсвтует названию солюшенов

В идеале внедрить AsyncApi

По всем пунктам готов к обсуждению и расшифровке.

@Alex-Grigorichev
Copy link
Contributor

На удаление Tracer в Api.Client уже есть задача #51

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