Created SQL for tables and scheme for them #4

Open
jezv wants to merge 6 commits from feature/database into develop
Owner
No description provided.
VolovikovAlexander left a comment
First-time contributor

Нужно дорабатывать

Нужно дорабатывать
@ -0,0 +14,4 @@
id UUID PRIMARY KEY DEFAULT gen_random_uuid(),
name VARCHAR(255) NOT NULL CHECK (char_length(name) >= 4),
phone VARCHAR(20) NOT NULL CHECK (phone ~* '^\+[0-9]{9}$'),
corp_id UUID NOT NULL REFERENCES corporations(id) ON DELETE CASCADE
First-time contributor

Нет. У нас один сотрудник может работать с несколкими организациями (филиалами). Тут связь должна быть Многие ко многим

Нет. У нас один сотрудник может работать с несколкими организациями (филиалами). Тут связь должна быть `Многие ко многим`
jezv changed title from WIP: Created SQL for tables and scheme for them to Created SQL for tables and scheme for them 2026-03-06 16:00:19 +08:00
Author
Owner

Переписал на EF который храниться и работает в PersonalAccount.Data.
В этом же проекте реализовал Seeder (который генерирует случайные данные)
А также для миграции реализовал AppliacationContextFactory.

Вся миграция лежит в PersonalAccount.Data/Migrations - туда не стоит смотреть!

По новому заданию:

  1. реализовал модель Settings внутри Corporation, так, что она сериализуется в JSON автоматом и обратно.
  2. реализовал DTO модели описанные в задании
Переписал на EF который храниться и работает в PersonalAccount.Data. В этом же проекте реализовал Seeder (который генерирует случайные данные) А также для миграции реализовал AppliacationContextFactory. Вся миграция лежит в PersonalAccount.Data/Migrations - туда не стоит смотреть! **По новому заданию**: 1. реализовал модель Settings внутри Corporation, так, что она сериализуется в JSON автоматом и обратно. 2. реализовал DTO модели описанные в задании
VolovikovAlexander left a comment
First-time contributor

Тестов нет. Репозитория / сервиса для построения отчетов я не нашел в Вашем коде. Оофрмление кода - ужастное. Описание по проекту - нет (DESCRIPTION.md)

Тестов нет. Репозитория / сервиса для построения отчетов я не нашел в Вашем коде. Оофрмление кода - ужастное. Описание по проекту - нет (DESCRIPTION.md)
@ -0,0 +3,4 @@
namespace Src.PersonalAccount.Domain.Models;
public class SalesDTO: IId
First-time contributor

Где коментарии в код?

Где коментарии в код?
@ -29,0 +30,4 @@
/// <summary>
/// Компании в которых работает сотрудник
/// </summary>
[Required] public List<Corporation> Corp { get; set; } = null!;
First-time contributor

Нет. У нас должна быть связь многие ко многим

Нет. У нас должна быть связь `многие ко многим`
jezv marked this conversation as resolved
@ -19,0 +31,4 @@
/// Транзакция с которой что-то там начинается
/// </summary>
[Required]
public required Guid StartTransactionId { get; set; }
First-time contributor

У нас идет загрузка из журнала, а в журнале "длинное целое" , а не GUID

У нас идет загрузка из журнала, а в журнале "длинное целое" , а не GUID
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feature/database:feature/database
git switch feature/database

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch develop
git merge --no-ff feature/database
git switch feature/database
git rebase develop
git switch develop
git merge --ff-only feature/database
git switch feature/database
git rebase develop
git switch develop
git merge --no-ff feature/database
git switch develop
git merge --squash feature/database
git switch develop
git merge --ff-only feature/database
git switch develop
git merge feature/database
git push origin develop
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
jezv/VolovikovASPNET!4
No description provided.