search

glenngnng / pe

0 stars 0 forks source link

Possible to add duplicate persons #2

Open glenngnng opened 10 months ago

glenngnng commented 10 months ago

It is possible to add duplicate persons with just a different change to one parameter.

For instance: two adams are added with the only difference being the address being Singapor and Singaporee

image.png

Hence, the adding of duplicates may occur due to simple errors made by the user.

nus-pe-bot commented 9 months ago

Team's Response

Thank you for your suggestion! First of all, this bug should be considered as low severity because quoting the PE guidelines: "Low severity: A flaw that is unlikely to affect normal operations of the product. Appears only in very rare situations and causes a minor inconvenience only." To have 2 employees having the same name, phone, and email but different address is rare in nature.

Secondly, this bug is not in scope because quoting the PE guidelines: "NotInScope: It is a valid issue but not something the team should be penalized for e.g., it was not related to features delivered in v1.4 or lower priority than the work already doen in v1.4." We consider this bug as of low priority than what we have worked on because the bug rarely happens and does not affect the usage of the app in any way. In addition, quoting the PE guidelines: "NotInScope, except if fixing the feature flaw is essential for the app to be reasonably useful". We see this bug as not essential for the general usage of our app because it does not hinder how HR people use HRInsight in any way and therefore, classify this bug as not in scope.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]

## :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]