allyfern72 / pe

0 stars 0 forks source link

[UG] Confusing salary range #16

Open allyfern72 opened 2 years ago

allyfern72 commented 2 years ago

It is stated that salary cannot be negative, however, a range including negative values is given right after.

image.png

nus-se-script commented 2 years ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

It was a mistake, it should be a minimum of 0.01.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Range of Salary under Add Staff is incorrect

The allowable range specified for the salary is incorrect/not clear:

image.png

Does this mean the salary can range from --2,147,483,648 to 2,147,483,647?


[original: nus-cs2113-AY2122S2/pe-interim#22] [original labels: severity.Low type.DocumentationBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

It was a mistake, it should be a minimum of 0.01.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your reason]