search

flexibo / pe

0 stars 0 forks source link

Clearing does not clear data for Courses #4

Open flexibo opened 1 year ago

flexibo commented 1 year ago

The clear command clears for all the persons in the addressbook, but it does not clear the data in courses.json

Since the courses given is part of the sample data, the data in courses should be cleared as well.

From UG: image.png

From courses.json after clear command:

image.png

Full demo for clear command followed by course command:


video:https://raw.githubusercontent.com/flexibo/pe/main/files/ca027329-05fe-47e8-92f7-240f2295d77e.mp4

This is low severity and a feature flaw because it later mentioned that the clear command works only for addressbook as shown below, but this is still a noteworthy bug to mention as a feature flaw since there is no command to clear courses based on the UG (including the FAQ section) and DG.

image.png

nus-se-script commented 1 year ago

Team's Response

As our list of courses are fixed, we did not see a need to implement a feature to clear the list of courses.

Screenshot 2023-11-19 at 10.05.24 PM.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]