lil1n / pe

0 stars 0 forks source link

multiple input required to add an event #5

Open lil1n opened 1 year ago

lil1n commented 1 year ago

to add an event into MoneyMind, multiple inputs into MoneyMind is required:

  1. providing the event command
  2. providing the category for the event to be added

the need for 2 steps to add an event (or maybe even 3 or more steps if there was no category in the first place / don't remember what category is inside MoneyMind), is slightly tedious. Could maybe have an alternate command for adding event such that a one liner could do both adding an event and specifying it's category

nus-pe-script commented 1 year ago

Team's Response

If we add one more specifier for the category, it makes the format become more complex and it will not favor fast typists as they can type wrongly and have to type again the long command. Therefore, we separate the command into two inputs to make it more user-friendly.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: As stated in the module website: https://nus-cs2113-ay2223s2.github.io/website/admin/tp-constraints.html#constraint-typing-preferred

image.png

Hence, I still stand by my point that a one-shot command alternative should be provided. This would allow fast typist to accomplish their tasks faster

Additionally, with the command prompt (windows, not sure about other OS), user can easily use the up arrow key to re-enter their previously typed commands, and edit from there should they make any mistakes.