search

b123b123z / pe

0 stars 0 forks source link

Auto filter after assigning deadline #1

Open b123b123z opened 4 years ago

b123b123z commented 4 years ago

image.png

Steps to reproduce:

  1. Populate reimbursement list by adding negative values to home tab to at least 2 different people i.e.

add dt/24-Aug-2019 d/Printer ink c/Miscellaneous a/-3.50 p/Alex Yeoh add dt/24-Aug-2019 d/Printer ink c/Miscellaneous a/-3.50 p/David Li

  1. Go to reimbursements tab and set a deadline for one person, i.e.

deadline p/Alex Yeoh dt/19-Dec-2019

  1. The list is auto-filtered to show only Alex's reimbursement.

This can be irritating when I'm assigning deadlines to a large group of people as i have to keep typing in "back" after assigning a deadline to an individual.

nus-pe-bot commented 4 years ago

Team's Response

Hi, I understand your concern. But this is just a "feature flaw" instead of "functionality bug".

In our design, even if you don't use "back" to view the full list, you can continue to assign deadlines to other people if you can remember the person's name correctly. We design this "back" just in case you want to view and see the full list and the person's name.

We designed to filter out and specifically show the deadline out of the following considerations:

  1. There tend to be lots of reimbursements in this tab. If we don't filter out and show this, it's very hard for the treasurer to confirm and see that the deadline is indeed added to the correct reimbursement. So after thinking about the pros and cons, we decided to filter the reimbursement out and let the treasurer see and confirm he has done the correct operation.

  2. The workflow of the treasurer is normally like this: 1) some CCA member approaches the treasurer and requests reimbursement before some day 2) the treasurer use "find" command to locate this reimbursement for the member and view the detailed information. 3) add "deadline" and see clearly that the deadline has been correctly added 4) go back and view the full list We initially designed this deadline feature as you proposed. But later during milestone discussions and according our treasurer friends' feedback, we changed this deadline feature to filter out the specific date so that it is easier for the treasurer to confirm his operation.

Additionally, this is a suggestion and not a bug: Screenshot 2019-11-16 at 9.48.55 PM.png

Thanks for pointing this out!

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]

:question: Issue severity

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

Reason for disagreement: [replace this with your reason]

:question: Issue type

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

Reason for disagreement: [replace this with your reason]