nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Subjects Allowed Are Limited Compared To Subjects Present #806

Open nus-se-script opened 1 day ago

nus-se-script commented 1 day ago

Description

The user guide states that, and I quote,

UGTeach is a desktop app for managing your students' contacts that aims to empower Singapore-based undergraduate private tutors to efficiently manage payments and organize schedules.

However, undergraduate private tutors are also responsible for tutoring JC students which expands the subject to Computing, Further Mathematics, Knowledge and Inquiry and many more. Furthermore, such subjects can also be banded to H1, H2 and even H3.

This may cause such an application to be limited to primary school tutors, as even secondary school has their mathematics subject split into Additional Mathematics and Elementary Mathematics.

Screenshot 2024-11-15 at 5.17.51 PM.png


[original: nus-cs2103-AY2425S1/pe-interim#1136] [original labels: severity.Medium type.FunctionalityBug]

Feng1231 commented 1 day ago

Team's Response

Thank you for your response. We will mark this as a duplicate of #145.

As for v1.6, we feel that enforcing the constraint for subject to only take a certain value is at least better than of allowing for any strings to be valid for the subject so that user. Improvements / widening of this scope could be postponed to the future iterations, since the current iteration would be sufficient for our target user most of the time. Hence, the severity is likely low, as it is only in rare case where the app will not be sufficient for the target user.

We believe that the subjects that you have provided are niche, there is only a small population of students who takes those subjects, in general. For the subjects that we allow in UGTeach, it should suffice for most undergraduate tutors who provides private tuition.

As for the Additional Mathematics and Elementary Mathematics, we feel that as for the current iteration, combining them into Mathematics will not bring too much inconvenience to the user. They can still have a workaround and indicate the subject as Mathematics.

Of course, your suggestion is valuable to our team in terms of improving the user experience for UGTeach. This issue will be resolved at the same time when we resolve #145, by allowing for a more flexible subject validation.

Duplicate status (if any):

Duplicate of #145