ptejasv / pe

0 stars 0 forks source link

Cannot add more than one lecture per module #12

Open ptejasv opened 3 years ago

ptejasv commented 3 years ago

Only one lecture can be added per module. This seems to go against the aim of the app as almost every module has more than one lecture, making this feature less useful for the user.

Steps to reproduce: enter calendar lecture m/cs2113 s/07-10-2021 e/31-10-2021 and then calendar lecture m/cs2113 s/07-11-2021 e/31-11-2021

Expected: Successfully added 2 lectures to the module cs2113

Actual:

Screenshot 2021-11-12 at 5.15.40 PM.png

nus-pe-bot commented 3 years ago

Team's Response

This is how the lecture feature works. Only one lecture is allowed to be added for each module. A warning message has already been provided for this feature, so it should not hinder the user who is using the app. Feature-wise, adding multiple lectures to the same module is not supported at the moment. Most of the modules usually have one lecture a week so for the current iteration of the app, this is what we go by.

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]