OhJunMing / pe

0 stars 0 forks source link

Command Summary add delete #11

Open OhJunMing opened 3 years ago

OhJunMing commented 3 years ago

Consider dividing general add and general delete to add person, add meeting

It would be nice without user having to make an inference.

image.png

nus-pe-bot commented 3 years ago

Team's Response

The command summary is intended to provide a concise view of all the commands, so such a division could clutter up the view and may not be helpful. In the current state, we argue the message is quite clear.

The additional line break should make clear the distinction between the ...p and the ...m commands.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]

Add Contact

Add Meeting

Delete Contact

Delete Meeting

Using the above format would clutter, fully utilise the a table format