search

maze508 / pe

1 stars 0 forks source link

Error in DG Undo Sequence Diagram #20

Open maze508 opened 7 months ago

maze508 commented 7 months ago

Details

The sequence diagram references 2 methods called getCurrState and restoreState which are actually called getCurrentModelState and restoreModelState in code

image.png

nus-pe-script commented 7 months ago

Team's Response

Nice catch! This seems like a typo and does not affect the developer's understanding of the way that the undo command works. Hence, we recommend reducing the severity to VeryLow, as in the rest of the explanation and diagrams, the issue does not persist.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]