nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Should empty date of births be allowed? #18

Open nus-se-script opened 9 months ago

nus-se-script commented 9 months ago

Given that your target user profile is volunteers for animal shelters, it is very common for these pet shelters to have stray animals that we do not know the date of births. However, it is not allowed to add an animal without a date of birth, as replicated by the command below:

add n/ollie i/1111 g/male s/dog b/pomsky da/2023-11-17

Hmmm... Should empty date of birth fields be allowed for this case?

Tagging this as a medium severity level because it is very common for stray animals to exist in these shelters. However, not tagged as a high severity as users can put in placeholder date of births and continue using your product!


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

Blizzeracz commented 9 months ago

Team's Response

We recognized the DOB may not be accurate, but in our design implementation, we considered the DOB to be an estimation instead of a known value for stray animals.

However, it is a good comment and it could be a future enhancement, so we decided to put it in NotInScope instead of Rejected.

Duplicate status (if any):

--