search

E0735389 / pe

0 stars 0 forks source link

Duplicate `/note` allowed #29

Open E0735389 opened 5 months ago

E0735389 commented 5 months ago

type note 1 /note a /note b, the a will be silently discarded.

nus-pe-script commented 5 months ago

Team's Response

As per the user guide:

image.png

The format for using the Note command is clearly stated, and we did not specify that it is possible to add multiple notes at once when using the Note command.

However, I do believe that the suggested chain usage of Note command can be convenient, and if given the chance, I would like to implement this to enhance the Note command feature.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: The stated issue is not a feature improvement, it is a bug (reason: when the command format is invalid, the program should raise an error instead of silently throw away part of the command)

## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** It may cause **occasional** inconvenience to some user (e.g. the user may accidentally include `/note` into the note itself, and the first note is wiped without any particularly easy way to recover it). Which suits severity.Medium.