search

snss231 / pe

0 stars 0 forks source link

No command exists to edit reminders #4

Open snss231 opened 2 years ago

snss231 commented 2 years ago

Can be annoying for the user to need to retype all fields if he only needs to change one

nus-se-script commented 2 years ago

Team's Response

Reminders only have 2 fields, thus changing both at once would not be a problem for fast typers, especially when one of the fields is a date. Most reminder descriptions are usually one or two words, such as "meeting" or "sign contract", in the event that the description gets very long, the GUI allows copying and pasting. We also want to enforce the habit of checking all information to protect the user, so they can't make wrong changes such as changing the description but forgetting about the date.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Any decent product should support CRUD functions - missing the update part is definitely a huge oversight that should have been implemented.

I object to the dev teams assertion that forcing the users to type all the information again helps them to check all information and protect the user:. It doesn't make sense that the app should make the user type more stuff so they "enforce the habit of checking all information". The user may, while trying to change one field, accidentally change the other due to a typo. Mistakes are inevitable since this is a feature that would be used quite often, and the dev team should be helping the user to minimise mistakes instead "enforcing habits"