Allowed adding of NRIC starting with T for elderly

[nus-se-script]

• ContactMate is meant to keep track of elderly
• Adding of NRIC starting with T should not be allowed, even if it is a valid NRIC under the Singapore Government
• NRIC starting with the letter T are for people who are of age 24 and below
• Defeats the purpose of having this application built for elderly
• Cause inconvenience in data entry as this is not protected against

Steps to reproduce: add i/t0104626d n/jackie Doe p/98765432 e/johnd@example.com a/311, Clementi Ave 2, #02-25 c/7 t/wheelchairUser t/livingAlone

---

_{[original: nus-cs2103-AY2425S1/pe-interim#1576] [original labels: severity.Medium type.FeatureFlaw]}

[Zenkoh1]

Team's Response

We believe that this is low severity, as firstly, this will only occur in very rare situations and will not affect the product, for the following reasons.

There are two cases.

In the case where the user accidentally changes the letter from a S to a T, the user will get the input rejected (correctly) because of the checksum validation done. (eg. S5087089H is valid, but T5087089H is invalid, and will get rejected by ContactMate). Hence, this will never happen.

There is also the case where the user intentionally tries to input a valid NRIC starting with T. In this case, we believe that, at this stage of the project, we should be allowing more flexibility in the inputs of the NRIC. If the user goes out of their way to input an NRIC starting with T (not accidentally), we believe that we should allow it because users might be trying to use the application in a way where we did not even think about. Furthermore, having a valid NRIC starting with T is not harmful in our application, as it is still a valid one that is verifiable by the government, and does not cause any harm in the logic of our application.

If we want to impose further restrictions and less flexibility with NRICs (rejecting NRICs starting with T), this will require further discussion with the target user, which is not in the scope of the current project

Duplicate status (if any):

--