Tariq-droid / pe

0 stars 0 forks source link

Inability to assign deadline after adding task #10

Open Tariq-droid opened 1 year ago

Tariq-droid commented 1 year ago

You cannot assign a deadline to an existing task. This is a problem as deadlines change many times during the semester which will result in the student deleting the whole task and adding it again going against the value proposition to save time.

image.png

image.png

nus-se-script commented 1 year ago

Team's Response

Hi, we would like to clarify that you are able to edit a deadline for a deadline task.

However, you are seeing that error message because you are trying to edit a 'deadline' attribute for a simple task, which does not have a 'deadline' attribute. This is expected behavior because the type of task is different, and we decided not to allow a task to switch types through the edit command since it opens up the possibility for users to accidentally change the task type. We also decided not to support this mid switching of task type because it is highly unlikely for a simple task to become a deadline, or event in actual usage.

For example, in the usage you brought up, if the deadlines change for an assignment over the semester, our app would support that as the original entry for the assignment would ideally be a deadline type as it has an initial deadline, and the user can change that as they please. Basically, if you wanted to edit a deadline of a task, it is likely that the task already had a deadline in the first place. And that is the intended usage of our application.

However, we can concede that there may be very specific scenarios in which a simple task may have a deadline impose on it and that is certainly a feature we are open to adding to our application in the future. However, we do not find this more important than the current features we have implemented and as such, we find this to be not in scope.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]