search

jivesh / pe

0 stars 0 forks source link

Instructions for adding appointment in error message are unclear #4

Open jivesh opened 2 years ago

jivesh commented 2 years ago

Screenshot 2021-11-12 at 4.47.54 PM.png

Steps to reproduce

  1. Enter appt -a
  2. Observe the error message

Problem

Extraneous git after s/DATE_AND_TIME

nus-pe-bot commented 2 years ago

Team's Response

This is a cosmetic issue and it is unlikely to affect the normal operations of the users.

The example stated directly below the command message is also correct (without the git), and the User Guide is correct as well, hence this will not pose much of an issue to users who attempt this command.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: If the user (particularly a non-technical user) sees conflicted versions of 'correct' commands, it will cause them some confusion and inconvenience. So this definitely isn't a severity.VeryLow given that there is an actual effect on the user. severity.Low might be acceptable given that there's no significant damage that might be done, and there may only be inconvenience.