search

muraddurrani / pe

0 stars 0 forks source link

Attendance dates displayed could be more useful #4

Open muraddurrani opened 2 years ago

muraddurrani commented 2 years ago

The purpose of the app is to aid in administrative duties, but the attendance information shown could be more useful in achieving that goal:

image.png

The dates shown are only for the current day and previous days. If I were running a day care, I would be more concerned with information that concerns today and upcoming days rather than days that have already passed, in order to plan better.

For example, I the Transport portion is really useful because it lets me see what transport I have to arrange:

image.png

nus-pe-bot commented 2 years ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Lack of display for attendance

Only 6 fixed days are displayed in GUI, the full attendance of a month cannot be displayed unless using charge command charge 1 m/04-2022 c/200 while charge command itself is not directly related to "view all attendance of the month" as its name suggests. This might get users confused about how to view the attendance other than the 6 fixed days displayed

Screenshot 2022-04-16 at 2.38.17 PM.png

[original: nus-cs2103-AY2122S2/pe-interim#806] [original labels: severity.Medium type.FeatureFlaw]

Their Response to the 'Original' Bug

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

Thank you for your response. However, our intention was to only display 7-days as mentioned in the UG as well.

image.png

Currently, what we understand from your bug report is that there might not be a way to view the monthly attendance of the pets unless you use the charge command. We have considered this issue but it is not feasible to display 30 days of attendance in the GUI without causing the GUI to be cluttered. Even if we display the full 30 days, there might be cases where other users report that they want to view the previous month as well.

Additionally, it is not frequent for the users to want to view one whole month of attendance unless they are charging, as such this should be of low severity instead of medium severity.

Nevertheless, it is a good suggestion that we will look into but it is not within scope.

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: The original bug report seems to be concerning how one would go about checking attendance for a month, by command or otherwise. My bug report was concerned with the decision to display only past dates and the current date in the UI as opposed to future dates.

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: My report argues that it would be much more useful to display future dates in the UI than past dates. It does not require adding more elements to the UI, so clutter is not an issue. Users likely want to know how many dogs they should be expecting over the next few days because that information concerns many aspects of running a day care (inventory, capacity, expected revenue, etc.). The current display does not allow for that and there is no easy way to accomplish it otherwise.

The transport element of the UI is a perfect illustration of a good feature for daycare operators. It would not make much sense to display past pick ups and drop offs because its in the past. As such, you have displayed future pick ups and drop offs, which is important information. I think the same logic applies to attendance.