-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Выглядит интересно, завтра с компа гляну подробнее
close #67 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@FrediKats, а ERD перерисуешь? 😁
Вернул обратно Division и изменения. Вынес фейкер. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Если считаешь что штука с количеством вызовов не нужна, то можешь её выпилить вместе с ITestUniversitySystemApi
Co-authored-by: ronimizy <71693183+ronimizy@users.noreply.github.com>
Kudos, SonarCloud Quality Gate passed! |
refactor: decompose division
Внезапно оказалось, что текущая реализация связей немного не работает. Сущность Division должна связывать множество групп в рамках одного предмета. Но это объединение не учебных групп. В системе явно не хватало сущности StudyGroupSubject - это как раз то, что описывает, что у определённой группы есть определённый предмет (и множество преподавателей). Это и есть единица, которую можно объединять в Division.
Это первый шаг к #38