search

joshuan98 / pe

0 stars 0 forks source link

Repeated messages on startup #19

Open joshuan98 opened 1 year ago

joshuan98 commented 1 year ago 
Please check that the format of the link provided is correct if there are missing modules or lessons.
Please visit https://ay2223s1-cs2113-f11-3.github.io/tp/UserGuide.html#import-a-timetable-import

The above message is repeated between semesters.

image.png

nus-pe-bot commented 1 year ago

Team's Response

This is the intended implementation to remind the user which modules are in their respective lessons. The repeated information also serves as dividers for users to easily identify the different semesters.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: If the group means to have a divider, then they should provide a divider in the form of a line instead of repeated messages.

"This is the intended implementation to remind the user which modules are in their respective lessons." - This is already handled by the lines saying "Semester x timetable imported". Thus, I do not see the need to have repeated messages.