search

xuwen-ho / pe

0 stars 0 forks source link

Remove mtg command should indicate optionality through square brackets #18

Open xuwen-ho opened 5 months ago

xuwen-ho commented 5 months ago

image.png

It is mentioned m/ and time/ prefixes are optional, however, in command they are shown as mtg NAME m/MEETING_DESCRIPTION time/MEETING_TIME when they should be mtg NAME [m/MEETING_DESCRIPTION] [time/MEETING_TIME], with square brackets showing optionality when the user wishes to remove a meeting

nus-se-bot commented 5 months ago

Team's Response

Our command format given is for adding meetings and the meeting and timing parameters would not be optional in that case. It is only for removing meetings that nothing is specified which is stated in the points later on.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Thank you for your response. As you have stated in your response, sometimes meeting and timing parameters are specified while sometimes nothing is specified, which means it is optional and should be labeled with []. Hope this helps you.