brein62 / pe

0 stars 0 forks source link

Model -> Schedule in AddScheduleCommand sequence diagram #12

Open brein62 opened 1 year ago

brein62 commented 1 year ago

The addCca() method in the sequence diagram is defined in seedu.address.model.person.Schedule class, not Model class, so the :Model object should be changed to :Schedule instead.

Screenshot 2023-11-17 at 5.27.25 PM.png

nus-se-script commented 1 year ago

Team's Response

While this may cause issues for new developers, a simple search for the method or code trace would bring the developer to the schedule class. Thus, while it inconveniences them for a while, we believe its very minor and should constitute a low severity bug.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: [replace this with your explanation]