search

lamyuewei / pe

0 stars 0 forks source link

remove function (rm) doesnt work as in user guide #9

Open lamyuewei opened 4 years ago

lamyuewei commented 4 years ago

remove function (rm) doesnt work as in user guide.

rm only ask for 1 line input

yet, it is unclear if this 1 line of input actually does remove the dir, infact it requires a second line and is not stated in the user guide as it req user to input yes or no in another line.

image.png

image.png

nus-pe-bot commented 4 years ago

Team's Response

The screenshot taken was from the command summary section, which only shows you the format of the command. The rm (delete) command you have entered is indeed correct. What comes after is a prompt to confirm the deletion. This is clearly illustrated in the User Guide and the Developer Guide as well.

User Guide:

image.png

image.png

Developer Guide:

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I do not think this issue should be rejected because we are suppose to use single line command to perform operations and not have multiple lines to ask the user if they are certain about their chose of action according to the forum:

https://github.com/nus-cs2113-AY1920S2/forum/issues/65

image.png

Besides, I just want to add that this group has create methods for undo and redo. Which makes it even more pointless for them to ask the user whether they are certain of performing the rm command since they can always just undo it, provided that they ensure their undo and redo is really working.