search

striris / pe

0 stars 0 forks source link

cannot cancel future booking #8

Open striris opened 2 years ago

striris commented 2 years ago

image.png cannot cancel future booking

nus-pe-script commented 2 years ago

Team's Response

cancel p/NAME uses a direct match to cancel borrow entries.

However, listfb p/sam does a partial match on names.

In this case, cancel p/ sam i/1 did successfully remove the future borrowing.

The user may expect listfb p/sam to do direct matches. However, this should not be an issue as the user could always just be more verbose to get exact matches.

Severity has been changed to medium as it may affect operations. Input shown by tester is reasonable and may happen in reality.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]