search

ernestlim8 / pe

0 stars 0 forks source link

Find diagram #14

Open ernestlim8 opened 3 years ago

ernestlim8 commented 3 years ago

image.png

Under the if else for [at least one of the prefixes is present], the [none of the prefixes are provided/...] can be replaced with [else]

nus-se-bot commented 3 years ago

Team's Response

Since there were only two other possible scenarios, we decided to list them out to make it more explicit for the reader.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: It is common practice to use else instead of the second possibility as it improves readibility of the diagram while conveying the exact same message