search

nus-tic4002-AY2021S2 / pe-dev-response

0 stars 0 forks source link

Usecase Delete All person from group #51

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years 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.

Extension show that this function is supposed to remove people from that group and not deleting the person from that group.

Hence the statement is logically flaw.

delete all person in that group !== removing person from that group.

As I thought that this function would delete the person from addressbook belonging to the addressbook itself.

image.png

[original: nus-tic4002-AY2021S2/pe-interim#51] [original labels: severity.Low type.DocumentationBug]

tototto commented 3 years ago

Team's Response

The documentation description is correct. It clearly states that it will delete, hence remove all person from the group. It did not state that it will delete all person in that group from the App.

Duplicate status (if any):

--