UdhayaShan1 / pe

0 stars 0 forks source link

[DG] Wrong `add` sequence diagram #21

Open UdhayaShan1 opened 7 months ago

UdhayaShan1 commented 7 months ago

This diagram was copied from the delete but some remnants remain.

image.png

image.png

soc-se-bot commented 7 months ago

Team's Response

Yes parse("1") is a typo, should of very low severity. add John Doe is correct as we omitted less important details here.

This shows the correct input but in the UML Diagram, we only showed John Doe as it is too long. image.png

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: I do not think this is veryLow as that is reserved for purely cosmetic issues.

Here, we are using the wrong commands like parse INDEX for add is which is wrong. This should be at least Low.