search

evedaktyl / pe

0 stars 0 forks source link

NameContainsKeywordPredicate Not Shown in Sequential Diagram #14

Open evedaktyl opened 5 months ago

evedaktyl commented 5 months ago

image.png

nus-se-bot commented 5 months ago

Team's Response

We consider the implementation of NameContainsKeywordPredicate to be a lower level detail and not very necessary to understanding how the find command is used. Adding that class to the sequence diagram would only complicate it without providing much value, not unlike how the ParserUtil class is also not shown in the sequence diagram despite being involved in the whole process.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]