search

jinzihan2002 / pe

0 stars 0 forks source link

Application only caters to CEG students going on SEP, but this is not specified by UG #6

Open jinzihan2002 opened 2 weeks ago

jinzihan2002 commented 2 weeks ago

image.png The application currently hard-codes course mappings for only computing and computer engineering courses. However the User Guide's introduction does not constrain the application to CEG students. In fact, the only location CEG is mentioned in the UG is a single instance under the list schools section.

Thus, it is possible for a non-CEG student to attempt to use the application for exchange planning, and realise that non of the provided mappings apply for them.

nus-se-bot commented 1 week ago

Team's Response

Thank you for the feedback! This was mentioned in our DG, but not in our UG, since we took reference from the AB3's UG introduction style as seen below, the target user is not mentioned:

image.png

but we could add it in for our next iterations to make it clearer for new users who come across our software!

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: Thank you for the response. However, I believe the bug is still in scope, as while the AB3 UG does not explicitly mention a target user, its description as a "desktop app for managing contacts" is consistent with the application's functions, and it performs as expected for a command line contacts management app. In contrast, based on the current ExchangeCourseMapper UG and app, a user reading the UG and skimming through the commands could reasonably expect the app to function for all course mappings for SEP in Oceania, but the app only provides CEG course mappings in reality, thus making it less useful for the UG's stated users. Also, the end use will only have access to the UG and not the DG.

## :question: Issue type Team chose [`type.DocumentationBug`] Originally [`type.FeatureFlaw`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** I believe the severity of this bug is still medium, as the lack of a sufficiently clear target user in the UG could cause a more-than-minor inconvenience to potential non-CEG users, who may download and run the app, and realise it is not applicable to them.