search

iscyng / pe

0 stars 0 forks source link

No error prompt when inputting plural form of allergies #5

Open iscyng opened 5 months ago

iscyng commented 5 months ago

UG mentions that allergies must be in singular form and not plural (egg not eggs) but it does not differentiate in the app itself

add jehiqhr, 100, 100, eggs, lunch, www.eggs.com

Woo hoo chef! I have added this recipe to your recipe book: jehiqhr / 100 kcals / 100 mins / added on 2024-04-19 / url: www.eggs.com You now have 18 recipes in your recipe book. Keep adding some!

Storage client says: Successfully saved recipe book.

Screenshot 2024-04-19 at 16.47.42.jpg

nus-se-script commented 5 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

UG incorrect documentation for add command

UG says allergies must be singular tense for add command but allows me to input plural tense allergies

image.png

[original: nus-cs2113-AY2324S2/pe-interim#2246] [original labels: severity.Low type.DocumentationBug]

Their Response to the 'Original' Bug

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

Sure, some explanation could be given in the UG about why we recommend singular tense.

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 type Team chose [`type.DocumentationBug`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]