search

eesung00 / pe

0 stars 0 forks source link

Error message not appropriate #3

Open eesung00 opened 1 year ago

eesung00 commented 1 year ago

image.png

The app check for duplicates in terms of Minecraft user name but not the person. Multiple person names can be added.

nus-pe-bot commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Duplicates detection should base on person names

image.png

A single person can have many accounts. Duplicates detection with Minecraft user-id is inconvenient (e.g. a person with 10 accounts will need to record 10 entries in this app)

[original: nus-cs2103-AY2223S1/pe-interim#92] [original labels: type.FeatureFlaw severity.Medium]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Thank you for your suggestion. However, Minecraft usernames are globally unique, but human names are not. As such, we choose to use Minecraft usernames for duplicate detection.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: This bug is not a duplicate of the bug marked. This bug is mentioning about the inappropriate error message for its current implementation instead of a feature flaw.

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: The app detect duplicates with Minecraft username instead of persons. Hence, I think this is a bug in terms of giving an inappropriate error message.

:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: This is not a feature flaw as the bug reported by me is on the error message given.

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: [replace this with your explanation]