nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

BalanceCommand doesn't show any variables #1065

Open nus-se-script opened 6 months ago

nus-se-script commented 6 months 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.


Screenshot 2024-04-19 at 5.40.14 PM.png It is required to show the variables section for a class, even if it has no variables in the class, that section should be shown to be empty


[original: nus-cs2113-AY2324S2/pe-interim#1266] [original labels: severity.Medium type.DocumentationBug]

mukund1403 commented 6 months ago

Team's Response

We assume that by "variables" the tester means attributes. The tester has mentioned that "It is required to show the variables section for a class, even if it has no variables in the class, that section should be shown to be empty" . This is incorrect based on CS2113 UML diagram guidelines:

Screenshot 2024-04-22 115723.png

We have followed class diagram format which is circled in the screenshot above.

Duplicate status (if any):

--