zuohui48 / pe

0 stars 0 forks source link

Overlapping sessions even though the location is different #3

Open zuohui48 opened 1 year ago

zuohui48 commented 1 year ago

I should be able to add a new session at a different location even if the timing overlaps with another session. Different athletes might need different forms of training, eg running on the track, working out in the gym. Hence I should be able to add concurrent sessions at different locations to provide for different training needs for the athletes.

Steps to reproduce:

  1. enter "create-session n/Hall s/10-03-2022 10:00 to 10-03-2022 11:00 l/MPSH2" to command box
  2. enter "create-session n/field s/10-03-2022 10:00 to 11-03-2023 11:00 l/MPSH2" to command box -> error occurs

Screenshot 2023-04-14 at 3.10.37 PM.png

soc-se-bot commented 1 year 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 raising this.

However, we believe this is more of a preference/suggestion rather than a feature flaw. Also, I would like to begin by highlighting this is an app for coaches managing their training sessions and not for athletes.

We have designed this feature in the following way based on needs raised by several Coaches that we had interviewed.

The reason why we disallowed creating overlapping sessions is that a coach can only be at a single place and at a single time. Furthermore, a coach cannot conduct two training sessions at the same time.

Following this logic, if we do allow what you have suggested, it will be a feature flaw, because it violates our logic above.

To add on, what you suggested seems to be a schedule planner for individual athletes themselves rather than a schedule planner for the coach, whose job is to conduct training sessions.

Thank you once again for this suggestion however, implementing this feature would go against the purpose of our application.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]