nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Income should be numerical #1366

Open nus-pe-bot opened 5 days ago

nus-pe-bot commented 5 days ago

The current implementation permits only low, medium, high, or unspecified incomes for each individual person. This represents a diminished utility for insurance sales representatives as numerical income details may be required for e.g. qualification for certain insurance schemes. An implementation change without substantial effort would be to parse the field as a non-negative integer input, like age, but with a larger income bound.

image.png


[original: nus-cs2103-AY2425S1/pe-interim#1885] [original labels: type.FeatureFlaw severity.High]

ty4g1 commented 4 days ago

Team's Response

Changed severity to Low as it is only a minor inconvenience that doesn't pose an issue to the app's functionality.

This is intended so that adding and filtering is easier for our users, without having to worry about specific numbers. It can leave the none, low, medium, high income interpretations to the user, thus not bounding them by our definitions. Though we agree that this can be improved to be more flexible in future.

Duplicate status (if any):

--