GraceWang2322 / pe

0 stars 0 forks source link

Incomplete glossary #19

Open GraceWang2322 opened 2 years ago

GraceWang2322 commented 2 years ago

The glossary does not include definition of essential terms such as cost that is eluded to several times in the previous parts

image.png

nus-pe-bot commented 2 years ago

Team's Response

The 'Original' Bug

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

More glossary terms could be added.

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.


Although the application is meant for insurance agents, it would be nice to include glossary for terms such as "revenue" (what is this revenue meant for? Is this the amount claimed by the client?) and "claim"


[original: nus-cs2103-AY2122S1/pe-interim#3903] [original labels: severity.Low type.DocumentationBug]

Their Response to the 'Original' Bug

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

Screenshot 2021-11-15 at 2.33.47 PM.png

Revenue is in the glossary. The team also felt that claim is unambiguous.

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: Although both reports are about the glossary, I believe that revenue and cost are not the same concept.

Revenue is straightforward as it would be profit which is earnings - cost. However, it would be better if cost is explained in the glossary as cost can be interpreted differently by different insurance agents such as cost for servicing (sending gifts, buying food) or cost of business (hiring staff). This is important as it would affect whether the revenue is calculated correctly. The cost of business should be shared among all clients, while the cost of servicing should be the cost under the client serviced.


:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Same reason as why it is not a duplicate status.


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]