search

nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Definition of positive integer is inconsistent #3049

Open nus-se-script opened 11 months ago

nus-se-script commented 11 months ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

Description:

Definition of Positive Integer in UG is inconsistent, where some consider 0 as valid while others do not.

image.png

image.png

image.png

Justification for Severity:

Might be a rare input for a user, but the inconsistency would confuser the reader as they are unsure what would be considered valid, hence low.

System Information

Chip: Apple M2 OS: macOS Sonoma 14.0

[original: nus-cs2103-AY2324S1/pe-interim#2416] [original labels: severity.Low type.DocumentationBug]

miljyy commented 11 months ago

Team's Response

The salary parameter should only take in non-negative integers.

Duplicate status (if any):

--