tanveersingh10 / pe

0 stars 0 forks source link

User guide lacks some details #14

Open tanveersingh10 opened 7 months ago

tanveersingh10 commented 7 months ago

UG doesn't specify that you cannot add people with duplicate names. I feel this is not intuitive cos its possible that I meet 2 people called Ryan or something.

soc-se-bot commented 7 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Lack of explanation on uniqueness in contacts, companies and internships in UG

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.


Description

The application currently validates that you cannot have duplicate contacts, companies, and internships on the basis of the names of these entities. However, the UG does not seem to explicitly define this criteria.

Reason for severity

This may hinder the frequently, especially with contacts due to the fact that your target audience may have many Singaporean names where the first name and last name are the same. For instance, a user might want to add 3 Jerome Tan contacts in their application. They would then have to infer from the error message that the application requires you to change some fields, and eventually change the name. Though the consequences are not high, the frequency might render the severity as Medium.


[original: nus-cs2103-AY2324S1/pe-interim#1481] [original labels: type.DocumentationBug severity.Medium]

Their Response to the 'Original' Bug

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

Thanks for bringing this to our attention!

We would first like to address "They would then have to infer from the error message that the application requires you to change some fields, and eventually change the name" - our error message is very straightforward that the "This person already exists in the address book".

We agree that we should be written in our UG that we detect duplicates. However, we believe that this is of low severity, as duplicate names are really not that common in Singapore (e.g. surname, chinese name differs a lot), hence we believe that not including it does not affect a majority of readers.

And users can easily add their own identifiers (e.g. "John Doe from CS" and "John Doe from Poly").

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: [replace this with your explanation]