nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Adding a new remark to a group overwrites the old remark #4710

Open nus-se-script opened 11 months ago

nus-se-script commented 11 months ago

image.png

image.png

Steps to reproduce: remark g/CS2103T r/quiz tomorrow remark g/CS2103T r/test on friday

image.png find g/CS2103T

There should be a warning that adding a new remark will overwrite the old one so that the user will be aware of that and no data will be lost in the process.

It is also better to allow user to add on to the remarks as there can be lots of remark for a group


[original: nus-cs2103-AY2324S1/pe-interim#3797] [original labels: type.FunctionalityBug severity.Medium]

lerxuann commented 11 months ago

Team's Response

Regarding why the remark command will overwrite existing remarks instead of add on, it was clearly stated in the DG as one of the design considerations for the remark command. Though it may be troublesome if the user wants to keep the original remarks of the group, it may be confusing to edit the original if there are too many changes, and adding on remarks would make it impossible to change any outdated remarks.

image.png

In terms of data loss, we stated under the planned enhancements in our DG that we plan to have an undo/redo function which will help if the user accidentally overwrites any remark on accident.

image.png

In view of the above, we believe this is not a featureflaw (i.e., the feature is not 'complete'.). However, we acknowledge this should have been stated in the UG so users know the specifics of this command, and thus we have filed it under DocumentationBug of Low severity

Duplicate status (if any):

--