nus-pe-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
No details provided by team.
The 'Original' Bug
[The team marked this bug as a duplicate of the following bug]
Application does not check if user has picked the lesson slots that have conflicts in timing
Steps to reproduce:
add cs2030 s1
add cs2040 s1
pick cs2030 lab 10d
pick cs2040 lab 2a
Expected: Application to warn user that there is a conflict in timing.
Actual: The app allows the user to pick both timings, even though they clash.
The user guide should warn the user that the onus is on the user to check for clashes in lesson timings.
[original: nus-cs2103-AY2223S1/pe-interim#5036] [original labels: type.FeatureFlaw severity.Low]
Their Response to the 'Original' Bug
[This is the team's response to the above 'original' bug]
This seems like a suggestion for our feature instead of a bug because our UG only mentioned that you can pick tutorial slots and add modules, but nothing about checking whether the tutorial slots clash with each other.
We reject this suggestion because some people might choose to add in all potential lesson timings they want when planning for modules to take and timetable management. Hence, we decide to put down the feature of allowing users to put classes/tutorials at the same time. Furthermore, we have never mentioned the feature of checking for timetable clashes in our user guide or user story.
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)
- [ ] I disagree
Reason for disagreement: [replace this with your reason]
:question: Issue response
Team chose [response.Rejected]
- [ ] I disagree
Reason for disagreement: [replace this with your reason]
As you can see from the image, I was able to select multiple modules with similar lesson slots and the app does not warn me of potential clashes. I know the user can be in the midst of deciding their modules, but it is still important for the user to be able to get a warning in case there are any matches since unlike NUS mods, there is no GUI interface to tell me if there are any issues with mods chosen.