nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Can only schedule one meeting per user #780

Open nus-se-script opened 10 months ago

nus-se-script commented 10 months ago

Being only able to schedule one meeting is a relatively important feature flaw, since it is very much likely that meetings may be repeated or that the user may meet with the same contact multiple times. The user guide and developer guide do not provide justification, enhancements or workarounds to this feature flaw. This could be medium severity, but we choose low severity since you can work around this by using notes, even though this is not stated.


[original: nus-cs2103-AY2324S1/pe-interim#661] [original labels: severity.Low type.FeatureFlaw]

imkwokyong commented 10 months ago

Team's Response

We recognize this as a feature flaw; however we mark this bug as response.NotInScope since we think that it's fine to delay the implementation of this to future iterations.

Moreover, in the UG we explicitly stated that this feature only support one schedule per person at a time:

image.png

As per requirement for response.NotInScope :

image.png

Any further extension of this feature was, at the time, a relatively poor value proposition :

  1. Most notably, it would require tweaking of the UI to support arbitrary amounts of schedules (or at least some large number), which is a greater ask than referring to the implementation of Tag, as the UI space taken is much greater than Tag (which is just one token!), in addition to further code changes to a newly-added feature.
  2. The value gained for the user in comparison to features like Note (which, as you rightly pointed out, was flexible enough to partially supplant the need for this feature expansion!) was small that the team decided it was not a focus at the time.

As such, we believe this qualifies as NotInScope, given the trade-off in not pursuing feature extension.

Duplicate status (if any):

--