thortol / pe

0 stars 0 forks source link

Will be troublesome to update the age of all the pets #4

Open thortol opened 1 week ago

thortol commented 1 week ago

image.png

image.png

Current Behavior Paw petrol is meant for clinics to keep track of their owner and pet data. However, currently the application stores the age of the pets. This will be troublesome for users as after a year, they will have to manually update all of the pets age by 1.

Suggested Improvement Don't store age as a field. If required can store birthday instead.

nus-se-script commented 6 days ago

Team's Response

Thanks! Although we did consider this, we wanted the flexibility to allow for pet age e.g. dog years for users to decide.

This will be troublesome for users as after a year, they will have to manually update all of the pets age by 1.

In addition, such batch updates should not occur. Updates should only occur on a birthday (not after a year as suggested). It is troublesome to verify the age of the pets and if they are still present. This can instead be update on next visit. Birthdates for some pets may also not be known and can only be estimated.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]