nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Unable to edit reminders solely on the date or time #1340

Open nus-se-script opened 2 days ago

nus-se-script commented 2 days ago

Inputted commands : re 1 dt/2024-01-02 or re 1 dt/13:00

Expected behaviour : Users may expect to be able to edit just the date or time of 1st reminder in the list.

Actual behaviour : Error message display saying that the DateTime is not in the correct format.

Users may have last minute changes in their appointments with clients and may just want to do a quick edit of the reminder by just inputting the field that is changed instead of needing to type out the whole dateTime format.

Suggested solution : When users input the example command above, only edit the field with the value that is attached to the "dt/" prefix.

For eg : re 1 dt/2024-01-02 edits only the date in the 1st reminder to become 2024-01-02 re 1 dt/13:00 edits only the time in the 1st reminder to become 13:00

Screenshot 2024-11-15 at 5.21.42 PM.png

Screenshot 2024-11-15 at 5.24.33 PM.png


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

rubinnn commented 1 day ago

Team's Response

Thank you for pointing this out!

However, this issue is currently not in scope. For the current version of ClientHub, our focus is to implement the entire Reminder feature from scratch. While we agree that this is a valid optimization, it is not our focus for the current version as the current reminder edit feature is already doing what it is supposed to. Furthermore, when the user attempts to only edit DATE or TIME, the application will not crash but instead, fails gracefully and provides a specific error message.

Therefore, we believe that this issue is notInScope

Duplicate status (if any):

--