search

g-tejas / pe

0 stars 0 forks source link

Nonfunctional Requirements need to be updated #10

Open g-tejas opened 3 months ago

g-tejas commented 3 months ago

Description

This is not a duplicate of the other bug, as the other one is a bug with the usage of the command itself when adding "nusc" tutors (functionality related). This is for the documentation itself, specifically inside the non functional requirements section.

The NFR states that it should be up to date with faculty values, however it is not ("NUSC"). NUSC is a valid faculty, and new. Furthermore, there are two old faculties (USP and Yale-NUS) which do not take in new students any more.

Furthermore, the UG needs to update the "VAlid faculty values" list with the up to date faculties as well.

Screenshots

CleanShot 2024-04-19 at 17.29.41@2x.png

Suggestion

Update the documentation with the new NUSC faculty and the two old ones.

nus-se-bot commented 3 months ago

Team's Response

Thanks for raising this. However, the issue states that our NFR is incorrect when it is indeed correct - that we want up-to-date faculty values.

The actual bug, which is regarding the UG Valid Faculty Values and the app itself (which is mentioned here not as the problem, but as a suggestion), this issue will be taken note of and be updated in future implementations (beyond the module).

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]