Laba1 Base PIbd-22 Kalyshev Y V #1

Closed
Zyzf wants to merge 5 commits from Laba1 into main
Owner
No description provided.
Zyzf added 5 commits 2023-02-07 14:40:02 +04:00
Zyzf requested review from eegov 2023-02-07 14:40:27 +04:00
eegov approved these changes 2023-02-11 20:48:35 +04:00
eegov left a comment
Owner

Зачтено с понижением балла(ов)

Зачтено с понижением балла(ов)
@ -0,0 +102,4 @@
}
return null;
}
//private OrderViewModel AttachManufactureName(OrderViewModel model)
Owner

Закомментированного кода быть не должно

Закомментированного кода быть не должно
@ -0,0 +17,4 @@
{
public int Id { get; private set; }
public int ManufactureId { get; private set; }
public string ManufactureName { get; private set; } = string.Empty;
Owner

Вычисляемое поле, не хранится в памяти

Вычисляемое поле, не хранится в памяти
@ -0,0 +48,4 @@
{
return;
}
ManufactureId = model.ManufactureId;
Owner

Не требуется обновлять все данные, только статус

Не требуется обновлять все данные, только статус
eegov closed this pull request 2023-02-11 20:48:41 +04:00

Pull request closed

Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Zyzf/PIbd-22_Kalyshev_Y_V_BlacksmithWorkshop_Base#1
No description provided.