nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Adding a person with cap 100 is rejected #2289

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

image.png

According to the app, the max cap and min cap should not exceed 100, and my input value isn't more than 100. When I only input 100, it succeeds, but not 100.00000000000000000000000000000000000000000000000000000000000000000000000000000000000 I should be expecting this command to succeed.

steps to reproduce:

add n/John Doe p/98765432 e/johnd@example.com a/311, Clementi Ave 2, #02-25 c/3.50/100.00000000000000000000000000000000000000000000000000000000000000000000000000000000000 g/male u/Nanyang Polytechnic gd/05-2024 m/Computer Science ji/173296 jt/Software Engineer Intern t/rejected


[original: nus-cs2103-AY2223S1/pe-interim#2980] [original labels: severity.Medium type.FunctionalityBug]

maryjess commented 1 year ago

Team's Response

Thanks for spotting! However, we consider setting the severity to Low as CAP (Cumulative Average Point) values for universities (aligning with our target user) usually may not reach 100, the max value is usually 4 or 5 so this occurrence is very rare.

Duplicate status (if any):

--