search

Ng-YZ / pe

0 stars 0 forks source link

Plural form for MCs #1

Open Ng-YZ opened 1 year ago

Ng-YZ commented 1 year ago

image.png Small issue, but would be nice if singular/plural form is accounted for in example shown above.

nus-se-script commented 1 year ago

Team's Response

Similar issue, where checking for the number of modules or modular credits before printing will sole the issue

The 'Original' Bug

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

Another plural form issue in printed message

image.png Again, small problem but hope this can be accounted for if only 1 module in the data.

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

Their Response to the 'Original' Bug

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

Accepted for cosmetic issue

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