search

dendendenden04 / pe

0 stars 0 forks source link

The grammar when displaying staff working day #14

Open dendendenden04 opened 1 year ago

dendendenden04 commented 1 year ago

image.png

The grammar here seems a bit weird. Is it better to use working on Monday rather than working in Monday?

nus-pe-bot commented 1 year ago

Team's Response

This issue is the same as #1280 as it refers to the grammatical error.

The 'Original' Bug

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

grammatical error in add_staff message

image.png

second line should be "working on Sunday" instead of "working in Sunday".

[original: nus-cs2113-AY2223S2/pe-interim#993] [original labels: severity.VeryLow type.FunctionalityBug]

Their Response to the 'Original' Bug

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

We agree that this is a minor grammatical error. As referred to the image below, we also changed the bug type to Documentation Bug even though the error is in the Ui.

image.png

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]