NivethaUdayakumar / pe

0 stars 0 forks source link

Accepts future dates #2

Open NivethaUdayakumar opened 1 year ago

NivethaUdayakumar commented 1 year ago

Program accepts invalid future dates:

image.png

Should throw error message stating invalid date entered!

nus-pe-script commented 1 year ago

Team's Response

The user has the option to keep a record of both their paid and unpaid expenses in a single list. However, with the "future expense" feature, they can maintain a separate list for upcoming expenses that have not yet been paid and therefore have not affected their account balance. Additionally, the tool allows the user to record expenses that were due in the future but have already been paid as "current expenses" by specifying their original due date.

The 'Original' Bug

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

Future expense and expense

What is the purpose of having separate expense and future expense commands? image.png

Expenses can record past and future expenses:

image.png

Can just have the expenses command and remove future expenses commands altogether as they have same functionality for convenience.


[original: nus-cs2113-AY2223S2/pe-interim#2458] [original labels: severity.Medium type.FeatureFlaw]

Their Response to the 'Original' Bug

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

They serve different purposes, while the user may choose to track both their paid and unpaid expenses in one list, the future expense allows them to keep track of future unpaid expenses in a separate list, not having been deducted from their account balance.

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 response Team chose [`response.Rejected`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.Medium`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]