nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Command name not intuitive for tasks and person clearing. #5625

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

Task clearing is task purge, but address book clearing is clear. Command is not intuitive, as clear implies that both tasks and address-book will be cleared, since task clearing command is so specific. Perhaps address purge or addr purge would be a better, more user-friendly command. The command clear could be reimplemented to ALL data in the application.


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

yeppog commented 2 years ago

Team's Response

The purpose of task purge and clear is inherently different. Tasks require purging as we need a quick way to remove tasks when they become redundant. Removing all contacts would happen on an extreme case, and it is unlikely someone would clear contacts that frequently as opposed to tasks.

Since there is already a clear command that clears the AB list, and a task purge command, we see no need to merge the commands together. Having them independent can allow the user to choose which action they would like, rather than restrict the user to clear everything, when they only want to clear the list in AB.

This helps to reduce the number of total commands.

Duplicate status (if any):

--