nus-cs2103-AY2021S1 / pe-dev-response

0 stars 0 forks source link

Required feature requires connectivity to internet #2456

Open nus-se-bot opened 4 years ago

nus-se-bot commented 4 years ago

Acceptance Test bug: The feature does not pass the requirement to help CS students plan for graduation, as there is no check for qualifying for specialisation tracks. Hence it does not fully meet the requirements for managing the requriements for a CS undergraduate as mentioned in the User Guide.

Screenshot 2020-11-13 at 4.55.02 PM.png Requirements checking: Screenshot 2020-11-13 at 4.54.11 PM.png

Furthermore, many of the validation steps for preclusions, are likely a part of the NUSMODS Database api functionality.


[original: nus-cs2103-AY2021S1/pe-interim#2274]

yan-soon commented 4 years ago

Team's Response

Preface: the issue is unclear, but we identified 3 bugs within and have addressed them as such:

Required feature requires internet connectivity to function: (Rejected) Required feature works offline, as the required modules are stored as JSON files and not dependent on NUSMODS and internet connectivity. (Assignees: yan-soon)

Specialisation tracks: (Accepted, Low) We have decided to exclude this feature for this implementation of the project due to the time constraint, however it is a feature that we considered for future iterations. Furthermore, the app still allows you to search for and add specialisation modules (just that they are not tracked under required), thus it does not entirely blindside the specialisation track. (Assignees: everyone)

Preclusion: (Rejected) Our preclusion data is once again stored as JSON files, and is not dependent on NUSMODS and internet connectivity. (Assignees: yan-soon)

Duplicate status (if any):

--