nus-cs2103-AY2223S2 / pe-dev-response

1 stars 0 forks source link

Some minor naming mistake in the sequence diagram #1738

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year 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.


image.png

image.png

In the sequence diagrams above logic call deletePerson() method but in your code base, it should be deleteEmployee() method as shown below

image.png


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

panavdua commented 1 year ago

Team's Response

Thanks for pointing it out, but we feel the severity label could we VeryLow as it's a cosmetic issue and does not cause any major inconvenience for the users/readers

Duplicate status (if any):

--