search

ruishanteo / pe

0 stars 0 forks source link

Empty commands clearing all related tags #7

Open ruishanteo opened 9 months ago

ruishanteo commented 9 months ago

While inputting empty tags like a/ is convenient for users to clear all the affiliations of the person, erroneous usage leads to very easy removal of all the data which is not ideal for users.

A separate clear command could be implemented instead to prevent this situation.

nus-pe-script commented 9 months ago

Team's Response

We do not wish to saturate our application with many commands as it will hinder point 5 of our NFR:

image.png.

Also, we do not wish to reduce the functionality of our commands just because we want to prevent potential mistakes made by the user. That would be counterintuitive to the target audience of our application which are head nurses that we believe have some proficiency in their typing.

Nevertheless, we cannot deny that there is potential in implementing a command to clear affiliations in the future.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]