search

nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Salary's constraints can be looser #1330

Open nus-se-script opened 10 months ago

nus-se-script commented 10 months ago

I understand in the UG that salary should be in numerals but in real life salary can have decimal values eg 1450.50 due to various reasons like CPF contribution in Singapore. Hence I think it should be more flexible to allow decimal values. Screenshot 2023-11-17 162635.png

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

starrylight99 commented 10 months ago

Team's Response

It is intended as our UG mentioned only numerals are allowed, and what you provide is termed as decimal number instead. While the case you have provided may be valid, we do think that it has a extremely rare use case and we would rather fit the requirements of the average users where salary is often recorded as whole numbers

Duplicate status (if any):

--