songivan00 / pe

0 stars 0 forks source link

Add command too long #3

Open songivan00 opened 1 year ago

songivan00 commented 1 year ago

I believe that there are too many compulsory fields in an add command making the use of adding a contact inconvenient

soc-pe-bot commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Add command requires too many compulsory fields

The command requires too many inputs, 11 to be exact.

Screenshot 2022-11-11 at 4.24.36 PM.png


[original: nus-cs2103-AY2223S1/pe-interim#2410] [original labels: type.FeatureFlaw severity.Low]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

This is the only way to use the add command, as anything less than 11 would mean lesser fields (lesser info). A shortcut to get around this is available through the import command

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I believe that having to type so many information in one lines hinders the usability of the app, this makes adding a user each time very difficult and considering the user guide stated that it is suppose to be efficient and convenient, this add command does not provide that. On top of that, realising you made a mistake and needing to shift all the way back to the front of the command line is also extremely tedious.

While there is a tradeoff between the amount information and usability, I believe that a balance have to be struck and in this case having 11 compulsory fields really greatly affects usability.

Making some fields optional, could allow the users to choose which field they want to include and improve their experience.

Screenshot 2022-11-15 at 12.48.11 PM.png


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]