nus-pe-script
commented
5 months ago Team's Response
Fixing the issue of unclear expense/event indexing in #1898 will resolve this issue too
The 'Original' Bug
[The team marked this bug as a duplicate of the following bug]
delExEv and addExEv does not make clear about indexes.
Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.
It should state that the /i EXPENSE_INDEX for delExEv and /exi EXPENSE_INDEX is from the list command. Even with explanation it was very confusing as the output for both commands does not mention what was added or deleted, only the index. Using more examples showing the item in the list to tie in with the item involved in addExEv and delExEv command would help clear the confusion.
[original: nus-cs2113-AY2324S2/pe-interim#1680] [original labels: severity.Medium type.DocumentationBug]
Their Response to the 'Original' Bug
[This is the team's response to the above 'original' bug]
No details provided by team.
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)
- [x] I disagree
Reason for disagreement: Its in completely different sections of the UG and a mistake that could have been rectified easily as the delExEv is represented correctly but the addExEv right above is unclear.
## :question: Issue severity Team chose [`severity.Medium`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
Tying into the previous issue, the summary for delExEv is put more clearly here however it does not do the same for addExEv even if it is the same index from list command