Created SQL for tables and scheme for them #4
No reviewers
Labels
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
jezv/VolovikovASPNET!4
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "feature/database"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Нужно дорабатывать
@ -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Нет. У нас один сотрудник может работать с несколкими организациями (филиалами). Тут связь должна быть
Многие ко многимWIP: Created SQL for tables and scheme for themto Created SQL for tables and scheme for themПереписал на EF который храниться и работает в PersonalAccount.Data.
В этом же проекте реализовал Seeder (который генерирует случайные данные)
А также для миграции реализовал AppliacationContextFactory.
Вся миграция лежит в PersonalAccount.Data/Migrations - туда не стоит смотреть!
По новому заданию:
Тестов нет. Репозитория / сервиса для построения отчетов я не нашел в Вашем коде. Оофрмление кода - ужастное. Описание по проекту - нет (DESCRIPTION.md)
@ -0,0 +3,4 @@namespace Src.PersonalAccount.Domain.Models;public class SalesDTO: IIdГде коментарии в код?
@ -29,0 +30,4 @@/// <summary>/// Компании в которых работает сотрудник/// </summary>[Required] public List<Corporation> Corp { get; set; } = null!;Нет. У нас должна быть связь
многие ко многим@ -19,0 +31,4 @@/// Транзакция с которой что-то там начинается/// </summary>[Required]public required Guid StartTransactionId { get; set; }У нас идет загрузка из журнала, а в журнале "длинное целое" , а не GUID
View command line instructions
Checkout
From your project repository, check out a new branch and test the changes.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.