search

FestiveCat / pe

0 stars 0 forks source link

Unable to add an appointment in the past using appt-b #2

Open FestiveCat opened 2 years ago

FestiveCat commented 2 years ago

As provided on the module website, Such overzealous input blocking can be considered a type.FeatureFlaw.

This is indeed true as peforming this: appt-b 1 time/2020-02-29-00-00 produces an error because it is in the past.

nus-pe-script commented 2 years ago

Team's Response

We are being realistic as we do not want agents to create appointment dates in the past. An appointment should be a time in the future, realistically speaking.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Did not address how the module website said not to perform overzealous input blocking without warning:

Overzealous input validation: It is better to warn rather than to block when inputs are not compliant with the expected format, unless accepting such inputs can hinder the operations of the software. Allowing such flexibility can in turn allow the user to use the software in ways you didn't even anticipate while overzealous rejection of inputs can annoy the user:

This blocks instead of warns users