search

ngnigel99 / pe

0 stars 0 forks source link

Delete lesson does not include AM/PM #2

Open ngnigel99 opened 2 years ago

ngnigel99 commented 2 years ago

image.png

vs expected output

image.png

nus-pe-script commented 2 years ago

Team's Response

A duplicate of #1025. To reiterate our point:

The time format for our program has been updated to the 24-hour format, according to https://github.com/AY2122S1-CS2113T-W11-3/tp/pull/173. The documentation on our UG is not updated.

The 'Original' Bug

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

List lesson does not show AM/PM

image.png

PM is missing here, as compared to what was mentioned in the UG

image.png

[original: nus-cs2113-AY2122S1/pe-interim#1068] [original labels: severity.Medium type.FunctionalityBug]

Their Response to the 'Original' Bug

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

The time format for our program has been updated to the 24-hour format, according to https://github.com/AY2122S1-CS2113T-W11-3/tp/pull/173. The documentation on our UG is not updated.

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: They are separate commands.

:question: Issue type

Team chose [type.DocumentationBug] Originally [type.FunctionalityBug]

Reason for disagreement: image.png

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Medium]

Reason for disagreement: [replace this with your explanation]