danusan-s / pe

0 stars 0 forks source link

Recurring expenses not included in budget #10

Open danusan-s opened 4 days ago

danusan-s commented 4 days ago

I feel like it is only logical to include recurring costs in the expense information in the default list command. From what I can see, only the default list command shows a expense information relating to the budget. Maybe have a separate command to show the expense information instead of bundling it with the default list command.

image.png

nus-pe-bot commented 19 hours 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

Thank you for pointing out the issue. I am not really sure what the feature flaw here is.

Do you want the team to:

  1. Add the recurring list to the current list?
  2. Have detailed report when i do list /recurring just like the default list?
  3. Show which of the expenses is recurring inside the data report when users use the default list?
  4. Make the default list just show the list. no need report. have a separate function to show report

I will assume that you meant the 1st option since to me that feels like the highest severity out of the rest. However, this functionality is out of scope for the following reasons:

  1. As the user, if they were to put an expense as recurring, then this expense would only apply to new lists when it is generated. It doesn't make sense for the recurring expense to affect the previous months/lists that were already generated.
  2. We do acknowledge that there could be an option to add the recurring expense to all other months, if the user deems so. However, this would then be out of scope of v2.1 since this functionality to modify what "recurring" is meant to be, is of a lower priority for this version
  3. DG specified what recurring expenses is under Glossary. Hence, we have clearly outlined what is supported and what is not supported, making this "optional" functionality out of scope.

For why the severity is lowered to Low instead of Medium. The following are the reasons:

  1. The recurring tag works as per stated in the DG.
  2. If the users deem that this recurring expense should be part of their current month, they could just easily use the add command to do so. It is only a change of tags, which makes it a minor inconvenience, not a major one.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your reason]


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your reason]