Heart-of-N1 / pe

0 stars 0 forks source link

Program accepts dublicate inputs #4

Open Heart-of-N1 opened 4 months ago

Heart-of-N1 commented 4 months ago

image.png

As shown in the image above, the program allows for duplicate inputs which can give rise to confusion if users are using the program to plan their trips. An error message is expected here.

nus-se-script commented 4 months ago

Team's Response

We understand why users would want some kind of prompt to say that a duplicate activity is being added. We will consider including this improvement in future versions of the product but for now this is not something that we do not want to happen.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: As per the guidelines, an issue might be out of scope if the rectification is considered less important based on value/effort considerations. However, allowing duplicate entries directly affects the core functionality of a planning application, making it a high-value fix. Considering that travel plans involve reservations and financial transactions, duplicates could lead to significant user inconvenience and monetary losses. The software does not provide a suitable error message when duplicates are entered, nor does it prevent the user from creating these duplicates. The lack of feedback to the user, when a duplicate is entered, does not constitute a graceful failure, thereby not satisfying the conditions for the NotInScope classification. Duplicate checks are a basic part of the functionality of any planning tool, and not an additional feature or enhancement. The absence of such a check is a flaw in the basic operations of the software, which is expected to prevent and alert against such entries.


## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]