rickkoh / pe

0 stars 0 forks source link

Unable to assign tag to a newly created contact #6

Open rickkoh opened 6 months ago

rickkoh commented 6 months ago

Description

image.png

image.png

I followed the "Try Out Your First Commands" portion of the user guide but the application doesn't allow me to assign tags to the contact (Step 3 of the user guide).

This code has an error.

Steps to reproduce

Follow the user guide

  1. Reset the application by running clear.
  2. Add a new contact add n/Xavier Tan p/98765432 e/xavt@example.com a/Ang Mo Kio street 2, Block 123, #01-01
  3. Assign a new tag assign Xavier Tan t/VPD t/E-Flag to the newly created user
  4. Notice an error message pops up.

Expected Behaviour

The User Guide should perhaps teach the user how to create a new tag first from scratch.

Actual

User Guide assumes that the application has the exact same data and tags already created.

soc-se-bot commented 6 months ago

Team's Response

For the Try Out Your First Commands! section, it is meant to be completed based on the default initial data when the user first runs the EventBook, without running other commands such as clear. As it was placed at the start of the UG, it is expected that the normal user would run the commands here when they first run EventBook, and the commands would work as expected. Furthermore, we kept it to be just a small set of commands as this section is just for users to get a feel of the main functionalities of EventBook, hence why we did not include other commands such as creating new event tags.

Perhaps what our team could do is to include an additional line to let users know to not run any other commands which would change the data before going through this section. However, we find that it is not too big of an issue and hence feel that severity VeryLow would be more appropriate.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: Hi, thank you for taking the time to respond. Retrospectively, I understand that the 'Try Out Your First Commands!' section may be intended for when the application is in its default initial state. However, as you mentioned, I believe including an additional line to advise users not to run any other commands beforehand would help prevent further confusion. Sometimes, users may inadvertently run commands like "clear," which could delete the tag. Instead of simply instructing users not to run any other commands that would alter the data before proceeding with this section, I suggest adding an extra line to prompt users to ensure they have a tag created or to teach them how to create one. This addition would ensure a smoother user experience and align with the section's purpose!


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