search

JodyLorah / pe

0 stars 0 forks source link

Command in DG is invalid command #14

Open JodyLorah opened 3 years ago

JodyLorah commented 3 years ago

Command shown in this diagram (add n/David) is misleading as it is an invalid command

image.png

nus-pe-bot commented 3 years ago

Team's Response

The additional fields are omitted for diagram simplicity.

It is obvious that this is a simplified command to be used, as there is an explanation before the diagram that includes ... to represent other necessary fields.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Who knows that ... is used to represent other necessary fields. There are so many interpretations of the ..., it can be an autofill option, or even a dev testing shortcut so the devs can test the field easily without the need to key in everything else.

:question: Issue severity

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

Reason for disagreement: It is not a cosmetic issue since this can be an actual input that the reader chooses to input and would affect their usage of the app, though not greatly.