isaaceng7 / pe

0 stars 0 forks source link

Parser class diagram is not simplified #14

Open isaaceng7 opened 4 months ago

isaaceng7 commented 4 months ago

Parser class diagram is too complicated and not simple as it contains everything that is tied to that class, instead of making it simpler and including the important attributes.

image.png

soc-pe-bot commented 4 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]

Diagram too small

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.


image.png Need to zoom in a lot to see diagram


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

Their Response to the 'Original' Bug

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

No details provided by team.

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]


## :question: Issue severity Team chose [`severity.Medium`] Originally [`severity.VeryLow`] - [x] I disagree **Reason for disagreement:** Severity should be VeryLow instead of Medium. The diagram can still be seen after zooming in, despite its small size. This is also mentioned in the textbook: ![image.png](https://raw.githubusercontent.com/isaaceng7/pe/main/files/2148acf0-46d9-4002-9ff1-5f8a0d843770.png)