search

insafhere / pe

0 stars 0 forks source link

Unable to view timetable despite adding a module that does not have classes (As such, users may think they did not put in any modules as error message is wrong) #3

Open insafhere opened 1 year ago

insafhere commented 1 year ago

image.png

Error message could state that the existing module does not have classes or it could showcase an empty timetable since module(s) have already been added.

nus-se-script commented 1 year ago

Team's Response

Selected module does not have time table in the first place. Timetable is empty is equivalent to the fact that selected modules do not have classes. User can use the list command if they are unsure if they had added the module. Issue can be solved with mc added being displayed.

The 'Original' Bug

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

No indication of MC's added

Both the ways to see the entire timetable you have planned have no indication of the number of mc's currently entered into the timetable, making it more difficult to plan out.

Screenshot 2022-11-11 at 5.04.04 PM.png

[original: nus-cs2113-AY2223S1/pe-interim#758] [original labels: type.FeatureFlaw severity.Medium]

Their Response to the 'Original' Bug

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

Since standard modules are 4 MCs, we find that students are typically able to track module credit information themselves. If they are unsure of how many MC a given module is, they can use the info command to find out.

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: This is a separate issue. It does not address the concern mentioned in my issue explanation.

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I disagree with your explanation as User would be unsure that their module have no classes if it is not stated clearly hence an error message is necessary.

Error message could state that the existing module does not have classes or it could showcase an empty timetable since module(s) have already been added.

:question: Issue severity

Team chose [severity.Low] Originally [severity.High]

Reason for disagreement: [replace this with your explanation]