CXIA17 / pe

0 stars 0 forks source link

Error in expiry date in update feature #7

Open CXIA17 opened 2 months ago

CXIA17 commented 2 months ago

When updating expiry date, if you input a date that was earlier than today, it will be saved. However, when you are setting expiry date, this will not be allowed. This creates some inconsistency.

Screenshot 2024-04-19 at 5.11.44 PM.png

Screenshot 2024-04-19 at 5.12.56 PM.png

nus-pe-bot commented 2 months ago

Team's Response

This is fair, and is indeed an inconsistency with respect to the way our application handles expiration dates.

However, we also believe that this is not something that a normal user will typically do.

Perhaps there might be some instances where a user may need to backdate expiration dates of items in their inventory? Though we cannot fathom why they would do so, we give the benefit of the doubt here.

Despite this, we believe that such occurrences are generally very rare, and have little to no impact on the functionality of our application whatsoever. Thus, we find that a Medium severity is not justified, and we have reduced the severity of this issue to Low instead.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: [replace this with your explanation]