search

muraddurrani / pe

0 stars 0 forks source link

Specifying pick-up and drop-off in the past does not result in anything #3

Open muraddurrani opened 2 years ago

muraddurrani commented 2 years ago

I am allowed to specify pick-up and drop-off times in the past but it does not cause any changes in the UI that reflect those entries.

If it was for record-keeping, I think there should be a command that lets me view past pick-ups and drop-offs for a dog. If its only for future use, I don't think it makes much sense to allow pick ups and drop offs in the past.

nus-se-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 report is about viewing attendance for days outside of the 6 displayed dates. My report is about being able to create past pick-ups and drop-offs without being able to view them.

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: It does not seem to me that the bug reported is not in scope. I think rectifying the issue is important because it is one of the primary features of the application, and the feature is slightly incomplete if the case where pick-ups and drop-offs are made in the past are not handled for.