search

nus-cs2103-AY2122S2 / pe-dev-response

0 stars 0 forks source link

Incorrect glossary #3226

Open nus-pe-bot opened 2 years ago

nus-pe-bot commented 2 years ago

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.

Screen Shot 2022-04-16 at 3.37.25 PM.png

"Person" term in the glossary is used sparingly in the DG. The term "Player" is used much more and isn't included in the glossary.

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

DALIN-Prog commented 2 years ago

Team's Response

'Player' is not ambiguous in this case as the DG/UG has defined that this application is for high school basketball team trainers to manage player's data and schedule. As such player is easily understood to be basketball players.

'Person' was defined as we used Person class in tP as such, we felt there was a need for 'Person' to be defined.

Duplicate status (if any):

--