Сделать рефактор Teaching

[lipa44]

Проект Web.Dtos

Что мы имеем

• GroupTeachersResponseDto - dto для возврата из контроллера / Query
• CreateSubjectTeacherRequestDto - dto для передачи в контроллер / Command
• SubjectTeachersDto - dto для возврата из контроллера / Query
• TeacherDto - обычная dto учителя / Query

Что я предлагаю

• Перенести SubjectTeacherCreateArgs куда-нибудь ближе к Commands, и подальше от Queries
• TeacherDto, GroupTeachersDto и SubjectTeachersDto - подумать, насколько нам нужно разделение учителей на 3 dto и решить, оставляем или нет
• Решить, нужно ли нам делать dto для возврата из контроллера, и если да, то заимплементить для всех остальных типов, а не только для учителей. И желательно вынести все dto как респонс тип контроллеров в отдельную папку, чтобы не перемешивать с обычными

Проект Web.Api

Что мы имеем

• Запросы перемешались с командами, кажется, этот репо для доступа к данным, а не для их изменения

Что я предлагаю

• Перенести команды из этого репозитория в тот, который отвечает за команды
• Заменить тип HTTP запроса у GetUserTeacherTypeForSubject и IsUserHasTeacherPermissionForSubject на Get (прочему-то стоит Post)

[FrediKats]

репо для доступа к данным, а не для их изменения

Проблема в том, что если это утверждение будет истинным, то это значит, что у нас нет возможности менять данные в базе. Этот сервис является корневым и другие сервисы тянут из него инфу. Если мы не будем иметь тут API для обновления, то никакой сервис не сможет обновить данные о студентах.

[lipa44]

репо для доступа к данным, а не для их изменения

Проблема в том, что если это утверждение будет истинным, то это значит, что у нас нет возможности менять данные в базе. Этот сервис является корневым и другие сервисы тянут из него инфу. Если мы не будем иметь тут API для обновления, то никакой сервис не сможет обновить данные о студентах.

Справедливо, с разделением репо я немного погорячился.