khor-jingqian / pe

0 stars 0 forks source link

Implementation for planner is unneccesary long #18

Open khor-jingqian opened 3 years ago

khor-jingqian commented 3 years ago

You give a step by step walkthrough then a sequence diagram then an activity diagram to reiterate the same steps. I think this can be cut down by alot.

nus-se-bot commented 3 years ago

Team's Response

The sequence diagram given only show the execution of a planner goto today command. However, the activity diagram given shows the executions of all the different types of planner commands, for example switching to planner tab, goto today/tomorrow and goto specific date/month. Hence, I do not think that it is reiterating the same steps.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

[ ] I disagree

Reason for disagreement: [replace this with your explanation]