gekhro / pe

0 stars 0 forks source link

Issue with discharge status #6

Open gekhro opened 1 year ago

gekhro commented 1 year ago

Patients that are discharged in the past are still able to be in the hospital wards and take up capacity there.

nus-pe-script commented 1 year ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

No details provided by team.

The 'Original' Bug

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

Error message requires future date-time, but past date-times are valid

The error message for an invalid date-time requires a "valid future date-time of the format...", however, you are able to input past date-times.

Screenshot 2023-04-14 at 2.46.37 PM.png

Screenshot 2023-04-14 at 2.47.05 PM.png


[original: nus-cs2103-AY2223S2/pe-interim#3492] [original labels: type.FunctionalityBug severity.Medium]

Their Response to the 'Original' Bug

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

Currently we allow adding past date-times. We have mentioned this behavior in our DG under Planned Enhancements. Hence, we consider this out of scope of our current product.

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 reason]


## :question: Issue response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your reason]