search

bryanwee023 / pe

0 stars 0 forks source link

Items expiring in 7 days not flagged as expiring #6

Open bryanwee023 opened 3 years ago

bryanwee023 commented 3 years ago

This is definitely a nitpick. When an item expires in a day, I would think it'll expire tomorrow. Hence, Milk (below) should expire in 7 days (today's the 12th). but it is not flagged as expiring.

image.png

UG states that it'll flag within 7 days from today's date.

image.png

nus-pe-bot commented 2 years ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Ingredient with expiry date 7 days from the current date is not tagged as expiring

As mentioned in the user guide:

image.png

To reproduce:

  1. The current date is 12-11-2021
  2. Edit an ingredient with an expiry date 7 days after the current date: edit ingredient 1 -e 19-11-2021

Expected:

  • The ingredient is tagged with expiring

Actual:

  • The ingredient is not tagged

image.png

Perhaps it would help to give an example to make it clearer to users

[original: nus-cs2103-AY2122S1/pe-interim#4928] [original labels: severity.Low type.FunctionalityBug]

Their Response to the 'Original' Bug

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

This is a documentation bug, it should have been "within 7 days of the current date". Good spot!

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: [replace this with your explanation]