nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Unable to support multiple notes, overrides previous ones #4740

Open nus-se-script opened 7 months ago

nus-se-script commented 7 months ago

Steps to reproduce:

  1. Give a note to a contact.
  2. Give another note to the contact.

Expected: It shows a warning that the contact already has a note, or it adds both notes to the contact.

Observed: The previous note becomes overriden. Users would be unable to supply multiple notes to the same contact, with possibly different deadlines.


[original: nus-cs2103-AY2324S2/pe-interim#4153] [original labels: severity.Medium type.FeatureFlaw]

jannaleong commented 7 months ago

Team's Response

While yes, it would be good to add multiple notes for the same contact, our intention for v1.4 is to add a note. This is specified in our user guide, saying "Adding a note" in our header and description of the note command.

image.png

As we prioritised ensuring that the current note command can add a note, change a note and remove the note without functionality bugs, adding multiple notes wasn't something we intended to implemented in v1.4.

Furthermore as this an address book and not a note taking app(would be morphing the product if it was), being able to handle the storage and display of multiple notes was not our priority. Even without handling multiple notes, the feature already provides significant value to the users by allowing them to capture additional details regarding the contact. Thus, while handling multiple notes would be a good enhancement, we did not feel like it was of high priority compared to value that has already been added by the existing implementation of note.

But, we appreciate your suggestion and will consider it for future iterations.

Duplicate status (if any):

--