search

conradsoon / pe

0 stars 0 forks source link

Incorrect error message for `addModule` when multiple `m/` fields specified #9

Open conradsoon opened 11 months ago

conradsoon commented 11 months ago

image.png

addModule m/Booboooboo  m/Blahblah

Running addModule with multiple m/ fields returns an incorrect error message.

In this case, there are no non-alphanumeric characters being used, but the app wrongly declares it otherwise.

Correct error message should be something along the lines of multiple m/ tags used, please only use one!.

nus-se-bot commented 11 months ago

Team's Response

Disputed Severity: Medium => Low

Reason: While the bug report itself is certainly valid, I do not think this warrants a medium severity.

The section under addModule in the UG states that only ONE module be added at a time.

add.png

As such, I see little reason for users to try multiple m/ and this should be a relatively rare occurence, falling under the definition for Low severity - Appears only in very rare situations and causes a minor inconvenience only.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]