Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix PED #317

Merged
merged 4 commits into from
Nov 12, 2024
Merged

Fix PED #317

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
66ae6f1
docs: Add invalid use case for list
Rachael-Chan Nov 12, 2024
98ff29d
Merge branch 'master' into PED-Fixes
Rachael-Chan Nov 12, 2024
d696356
docs: Update duplicate tags handling
Rachael-Chan Nov 12, 2024
2e3cab3
docs: Fix sentence structure of tags in known issue
Rachael-Chan Nov 12, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions docs/DeveloperGuide.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -295,6 +295,12 @@ Priorities: High (must have) - `* * *`, Medium (nice to have) - `* *`, Low (unli

Use case ends.

* 2b. The given list command has extraneous parameters.

* 2b1. SocialBook shows an error message.

Use case ends.

##### **Use case: UC02 - Add a contact**

**MSS**
Expand Down
2 changes: 2 additions & 0 deletions docs/UserGuide.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,6 +92,7 @@ SocialBook is a **desktop app designed for social workers in Singapore, optimize
* You can include hyphens and spaces as necessary between words for tags that are multiple words long!
* To indicate no tags for a contact, you can `add` a contact without any `t/` prefixes.
* Take note that `add`ing a contact with a `t/` prefix followed by whitespace is not supported. Omit the `t/` tag for contacts without tags.
* Take note that tags are case sensitive i.e `financial` and `FINANCIAL` would not be considered as duplicates.

### Remarks
* Remarks are optional for contacts, and are denoted by the `r/` prefix.
Expand Down Expand Up @@ -360,6 +361,7 @@ Furthermore, certain edits can cause the SocialBook to behave in unexpected ways
This is due to the way that parsing is handled: any text matching special prefixes will be identified as such.
To work around this, please prepend a `_`, or any other accepted special character, in front of any input fields that contain a special prefix.
5. Entering an invalid emergency contact will produce the same error message as an invalid phone number. When receiving an error message about phone numbers, check both the `ec/` and `p/` field, if both are present.
6. Tags with the same name but with different cases are considered two different tags i.e `free` and `FREE` are not considered duplicates.

--------------------------------------------------------------------------------------------------------------------

Expand Down