sohenze / pe

0 stars 0 forks source link

Edit command: specifying d/ without input parameters #3

Open sohenze opened 1 year ago

sohenze commented 1 year ago

Steps to reproduce:

  1. add c/Bytedance r/Backend Engineer s/Online Assessment d/24-Sep-2022 15:00 t/high pay l/Java l/Python
  2. edit 7 d/

Expected Result: UG should say that specifying d/ without any parameters will remove the date and time but it does not or Error/warning message to the user

Actual Result: Application's date and time is removed without any prior notice to the user

Before edit command:

image.png

After edit command:

image.png

Users may accidentally input d/ without any inputs and not know that doing so would remove the date and time from the application

nus-se-script commented 1 year ago

Team's Response

missing documentation in UG

Items for the Tester to Verify

:question: Issue type

Team chose [type.DocumentationBug] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: image.png

The edit command is a command that is expected to be used quite frequently by users as they would want to update their internship progress frequently. This bug occurs in a command that is used more than just "very rare situations".

After the edit is performed and the date is removed, the resulting command result or system message does not mention that the date has been removed. Since WorkBook sorts internships based on date, the position of the edited internship in the list would be changed as well. Both of this would likely result in the user being completely unaware that the date has been removed. As such, I do believe that this is more than just a "minor inconvenience" as the user may not even know that something has went wrong.

The bug occurs more frequently and causes more inconvenience than what severity.Low describes.