soc-pe-bot
commented
2 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]
UML Diagram for Model Does Not Include Year in Person
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.
In this image, the attributes of Person can be seen as Id, Name, Phone, Address, Tag and Email.
However, in the code base, the Person class also has a dependency on YearJoined which is not captured in the diagram.
[original: nus-cs2103-AY2324S2/pe-interim#4906] [original labels: severity.Low type.DocumentationBug]
Their Response to the 'Original' Bug
[This is the team's response to the above 'original' bug]
Thank you for your feedback!
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)
- [ ] I disagree
Reason for disagreement: [replace this with your explanation]
In the first Model UML Diagram,
The Person is related to ID, Name, Phone, Address, Tag and Email.
In the second Model provided just below,
The Person is related to more classes.
This is confusing as the reader will be uncertain as to which is the actual case.