Kate/ISEbd-21_Vasina_E.S._ComputersShop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

ISEbd-21 Vasina E. S. lab_4 #6

Closed
Kate wants to merge 6 commits from Lab_4 into lab_3
pull from: Lab_4
merge into: Kate:lab_3
Conversation 1 Commits 6 Files Changed 38 +2476 -16
Kate commented 2023-05-10 01:17:55 +04:00
Owner
Copy Link
No description provided.
Kate added 6 commits 2023-05-10 01:17:56 +04:00 
Reviewed-on: http://student.git.athene.tech/Kate/ISEbd-21_Vasina_E.S._ComputersShop/pulls/3
Reviewed-on: http://student.git.athene.tech/Kate/ISEbd-21_Vasina_E.S._ComputersShop/pulls/4
Kate requested review from eegov 2023-05-10 01:18:04 +04:00
eegov approved these changes 2023-05-27 21:11:59 +04:00
eegov left a comment
Collaborator
Copy Link

Не во все реализации OrderStorage были внесены требуемые изменения

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

Не во все реализации OrderStorage были внесены требуемые изменения Зачтено с понижением балла(ов)
ComputersShop/ComputersShop/FormReportComputerComponents.Designer.cs
@ -0,0 +103,4 @@
this.Controls.Add(this.buttonSaveToExcel);
this.Controls.Add(this.dataGridView);
this.Name = "FormReportComputerComponents";
this.Text = "FormReportComputerComponents";
eegov commented 2023-05-27 21:10:44 +04:00
Collaborator
Copy Link

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

Заголовок формы оформлен неверно
ComputersShop/ComputersShopBusinessLogic/OfficePackage/ReportLogic.cs
@ -0,0 +31,4 @@
/// <returns></returns>
public List<ReportComputerComponentViewModel> GetComputerComponent()
{
var components = _componentStorage.GetFullList();
eegov commented 2023-05-27 21:11:26 +04:00
Collaborator
Copy Link

При получении изделий, у них уже есть список компонент, отдельно его получать не требуется

При получении изделий, у них уже есть список компонент, отдельно его получать не требуется
eegov closed this pull request 2023-05-27 21:12:03 +04:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
eegov
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Kate/ISEbd-21_Vasina_E.S._ComputersShop#6
No description provided.
Delete Branch "Lab_4"

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?