ISEbd-22_Andrikhov_A.S. Lab work 1 BASE #1

Closed
asoc1al wants to merge 2 commits from lab1 into main
Owner
No description provided.
asoc1al added 2 commits 2024-05-26 12:52:32 +04:00
asoc1al requested review from eegov 2024-05-26 12:53:15 +04:00
eegov approved these changes 2024-06-02 16:11:47 +04:00
eegov left a comment
Collaborator

Проверено

Проверено
@ -0,0 +1,19 @@
<Project Sdk="Microsoft.NET.Sdk">
Collaborator

Название проекта не соответствует варианту

Название проекта не соответствует варианту
@ -0,0 +96,4 @@
Controls.Add(AddButton);
Controls.Add(DataGridView);
Name = "FormRepairs";
Text = "FormRepairs";
Collaborator

Заголовок формы оформлен неверно

Заголовок формы оформлен неверно
@ -0,0 +101,4 @@
try
{
var OperationResult = _orderLogic.TakeOrderInWork(CreateBindingModel(id));
Collaborator

Не требуется каждый раз передавать все данные заказа

Не требуется каждый раз передавать все данные заказа
@ -0,0 +48,4 @@
return true;
}
public bool TakeOrderInWork(OrderBindingModel Model)
Collaborator

Логику смены статуса можно было унифицировать

Логику смены статуса можно было унифицировать
@ -0,0 +51,4 @@
DateImplement = Model.DateImplement;
}
public OrderViewModel GetViewModel => new()
Collaborator

Название изделия нигде не заполняется

Название изделия нигде не заполняется
@ -0,0 +1,5 @@
namespace AutoWorkshopView;
Collaborator

Пустых классов быть не должно

Пустых классов быть не должно
eegov closed this pull request 2024-06-02 16:11:55 +04:00

Pull request closed

Sign in to join this conversation.
No reviewers
No Label
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: asoc1al/ISEbd-22_Andrikhov_A.S._AutoWorkshop#1
No description provided.