search

adam-ky / pe

0 stars 0 forks source link

Optional parameters for findclass not specified in square brackets in command summary #7

Open adam-ky opened 2 years ago

adam-ky commented 2 years ago

I notice some parameters for findc are optional, but they are not wrapped in square brackets to denote they are optional.

image.png

For example, in editc, the optional parameters are wrapped in square brackets

image.png

nus-pe-bot commented 2 years ago

Team's Response

For findc command, you can only search with one prefix at a time, hence there are no optional prefixes, and so not stated as optional in the summary.

Stating it as optional will in fact be wrong as you are not allowed to search with more than one prefix at a time.

Hence we reject this bug.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Optional parameters for find not specified in command summary

This is similar to issue #7

image.png

[original: nus-cs2103-AY2122S1/pe-interim#3149] [original labels: severity.VeryLow type.DocumentationBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

For find command, you can only search with one prefix at a time, hence there are no optional prefixes, and so not stated as optional in the summary.

Stating it as optional will in fact be wrong as you are not allowed to search with more than one prefix at a time.

Hence we reject this bug.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]