search

nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

lack of screenshots for manual testing delete command #5752

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 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.

Unable to see what the app would look like when the given commands have been entered, given I don't have access to the app now.

Becomes too wordy.

Screenshot 2021-11-12 173151.png

[original: nus-cs2103-AY2122S1/pe-interim#5738] [original labels: severity.VeryLow type.DocumentationBug]

chingh20 commented 2 years ago

Team's Response

Hi thanks for the input. The lack of access to the app is actually an artifiical constraint imposed due to the PE, so it should not be an issue for normal users. The expected response of the app is also written, and it isn't too long, so we do not think this is an issue. In fact, actually the GUI outcome of deleting an entry is probably exactly what one would expect, and we feel that adding more screenshots provides little value, so we did not want to clog up the UG unncessarily.

Duplicate status (if any):

--