search

UnwilledTangent / pe

0 stars 0 forks source link

Adding NRIC beginning with P should not be allowed! #6

Open UnwilledTangent opened 1 year ago

UnwilledTangent commented 1 year ago

image.png

Adding NRIC beginning with P should not be allowed, because there exists no such NRIC beginning with P.

nus-se-bot commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Adding NRIC beginning with T24 and above should not be allowed

image.png

Adding NRIC beginning with T24 and above should not be allowed, because this year is 2023, and 2024 babies and beyond have not been born yet.

[original: nus-cs2103-AY2223S2/pe-interim#336] [original labels: severity.VeryLow type.FeatureFlaw]

Their Response to the 'Original' Bug

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

Declared as NotInScope because it is fine to delay lower priority work until future iterations.

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: Thank you very much for marking this as a duplicate.

However, I would say that this current issue is different from the original bug.

For the original bug, I meant to point out that babies born in the future, such as in 2030, should not be added into the system. Thus, NRICs such as "T3012345A" should not be added.

For the current bug, I meant to point out that the prefix should only be that of "S" and "T", and not other typos such as "Z".

## :question: Issue response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]