damithc / foo

0 stars 0 forks source link

Delete every person from a group, use remove better? #111

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

[destination: nus-tic4002-AY2021S2/pe-moderation assignee: boydanderson labels: response.Rejected severity.Low team.3 tutorial.TIC4002-F18 type.DocumentationBug duplicate pending.0]

Summary of items to check:

Nothing to check :-)



Issue Description

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.


I thought it will delete all the persons assigned to a particular group, however, after test the command, it only remove all person from the group. So I think it is better to use remove all person from a group rather than delete, because delete sounds like you are going to delete the person from the app.


[original: Yiheng0410/pe#6]

Team's Response

No response provided.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Usecase Delete All person from group

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]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

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.

To Check [0 Items]

Nothing to check :-)