search

nus-cs2113-AY2324S1 / pe-dev-response

0 stars 0 forks source link

"list-activity" command accepts invalid flags #957

Open nus-pe-bot opened 12 months ago

nus-pe-bot commented 12 months ago

Per UG, the syntax of listing activities is "list-activity [-d]", see screenshot of UG: image.png

However, the "list-activity" command accepts additional flags, see screenshot: image.png This could confuse users

[original: nus-cs2113-AY2324S1/pe-interim#894] [original labels: type.FunctionalityBug severity.Low]

DaDevChia commented 12 months ago

Team's Response

The User Guide specifies the correct syntax and the documented features work as described. The user will only see the detailed list when using the -d flag and otherwise see the normal view. Therefore providing any other flag does not change the expected output and, thus won't confuse the user either.

The ability of our application to handle and tolerate extra flags or random characters without malfunctioning is a feature of resilience, not a bug. In terms of user experience, ignoring irrelevant flags and characters is more user-friendly than returning an error. This design choice prevents user frustration from minor command errors and allows the user to accomplish their tasks without unnecessary interruptions.

Duplicate status (if any):

--