search

wanghejin / pe

0 stars 0 forks source link

Unintuitive warning #6

Open wanghejin opened 10 months ago

wanghejin commented 10 months ago

UG specified that PRIORITY only considers the first letter of the input ( h for high, m for medium, l for low, or n for nil). For example, bprio 3 hlow sets the priority of buyer 3 to high.

But I found it unintuitive because user may accidentally input "h" in front of "low" but it is wrongly read as high instead of low image.png

nus-pe-bot commented 9 months ago

Team's Response

Hi thanks for the input! Yes we agree, however we have already put it as one of our planned enhancements in the DG at number 3; the command regex as we see it is probably slightly too flexible. Hence we reject this issue.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]