search

lwlshawn / pe

0 stars 0 forks source link

Serious error in documentation of delete command #16

Open lwlshawn opened 3 years ago

lwlshawn commented 3 years ago

image.png

The documentation for delete states that it will only delete the corresponding elderly's next visit, but for elderly with recurring visits, it will delete ALL the visits and not just the "next visit". I felt that there was a significant difference between the two, hence the high severity.

nus-pe-bot commented 2 years ago

Team's Response

It is not as significant of a documentation bug, since it would not hinder the app's usage in most cases. This could be mitigated by executing a visit command to add it back, so it would not make the app unusable, but only make it slightly more inconvenient for new users who may not be familiar with the app and this issue.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.Medium] Originally [severity.High]

Reason for disagreement: [replace this with your explanation]