search

jundatan / pe

1 stars 0 forks source link

Remind command can allow 1000001 #10

Open jundatan opened 1 year ago

jundatan commented 1 year ago

Remind command says the format should be upto 1000000 only. But allow 1000001.

Steps to reproduce: Enter the command remind 1000001

Expected: Error thrown that the days are bigger than 1000000

Actual: Shows the reminded list

image.png

nus-se-script commented 1 year ago

Team's Response

It is specified that the number of days should not go beyond 1000000 and if the user does input that value, the product can still work but there may be unforeseen issues that may occur. The number of days as an input is an overzealous input. Allowing the users to still enter a value above 1000000 creates flexibility for the users even if it does not guarantee that the product can still work normally, this is additional value for the users.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: There is no warning in the UG that the product can still work or anything in the UG.

image.png

The team cannot just assume that the user knows about this. If they have prove that they specified it somewhere, they should have given a screenshot of it rather than just stating that they specified it which i find it not reasonable at all.

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